Skip to content

Conversation

@simahawk
Copy link
Contributor

take over and fix #351.

Contains a small improvement on tests.

@OCA-git-bot
Copy link
Contributor

Hi @simahawk! Thank you very much for this contribution. As the addon you are improving does not have a declared maintainer, I take the opportunity to mention that you can consider adopting it. To do so, please read the maintainer role description, and, if interested, create a pull request to add your GitHub login to the maintainers key of the addon manifest.

Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@simahawk
Copy link
Contributor Author

the builds are all green, not sure why is not update here. Since this is the same as #351 that was approved, I merge.

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-366-by-simahawk-bump-patch, awaiting test results.

@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 3c97ddc. Thanks a lot for contributing to OCA. ❤️

@simahawk
Copy link
Contributor Author

/ocabot migration base_import_async

@OCA-git-bot OCA-git-bot added this to the 14.0 milestone May 25, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request May 25, 2022
9 tasks
@simahawk
Copy link
Contributor Author

/ocabot migration test_base_import_async

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants