Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIG] sale_order_product_recommendation_secondary_unit: Migration to 12.0 #1005

Conversation

chienandalu
Copy link
Member

cc @Tecnativa TT20511

@pedrobaeza pedrobaeza added this to the 12.0 milestone Nov 28, 2019
@pedrobaeza pedrobaeza requested a review from yajo November 28, 2019 20:05
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 28, 2019
54 tasks
@sergio-teruel
Copy link
Contributor

@chienandalu Can you increase coverage??

@pedrobaeza
Copy link
Member

It's true that seeing the diff coverage, this module is essentially not tested. You can refactor tests in base module as I showed you in the past for inheriting that one only adding the secondary unit part.

@chienandalu
Copy link
Member Author

Sure, I'll do that

@chienandalu chienandalu force-pushed the 12.0-mig-sale_order_product_recommendation_secondary_unit branch from 8b90985 to 2c98db4 Compare November 29, 2019 17:36
@pedrobaeza
Copy link
Member

Travis is red

@chienandalu chienandalu force-pushed the 12.0-mig-sale_order_product_recommendation_secondary_unit branch from 2c98db4 to 5787503 Compare December 2, 2019 10:59
@chienandalu chienandalu force-pushed the 12.0-mig-sale_order_product_recommendation_secondary_unit branch from 5787503 to ce1a3fc Compare December 2, 2019 11:44
@pedrobaeza
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-1005-by-pedrobaeza-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 2, 2019
Signed-off-by pedrobaeza
@OCA-git-bot OCA-git-bot merged commit ce1a3fc into OCA:12.0 Dec 2, 2019
@OCA-git-bot
Copy link
Contributor

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

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.

None yet

7 participants