-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
[14.0][MIG] purchase_request_operating_unit #348
[14.0][MIG] purchase_request_operating_unit #348
Conversation
ba68243
to
2a5b7e9
Compare
5cd0dab
to
e8caedb
Compare
CI is red, please check. |
@Saran440 can you check travis / rebase. |
@Saran440 I noticed the depend module was merged so I restarted Travis/Runbot and things look good there. I think all you need to do is squash the [UPD] and Translation commits to clean those up and then I will review. |
[FIX] remove reference to warehouse [ENH] purchase_request_operating_unit, purchase_rquest_procurement_operating_unit 9.0 purchase request operating unit - fixed travis (OCA#81) * [FIX] travis
…rchase_request_to_rfq_operating_unit
Currently translated at 12.5% (1 of 8 strings) Translation: operating-unit-12.0/operating-unit-12.0-purchase_request_operating_unit Translate-URL: https://translation.odoo-community.org/projects/operating-unit-12-0/operating-unit-12-0-purchase_request_operating_unit/pt/
8d7bb3f
to
59ef569
Compare
@kittiu @patrickrwilson Sorry for late, I fixed and rebase already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few cosmetic requested changes but no blockers, LGTM.
@@ -0,0 +1,30 @@ | |||
<?xml version="1.0" encoding="utf-8" ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line isn't needed anymore since v11. Not a blocker.
@@ -0,0 +1,30 @@ | |||
<?xml version="1.0" encoding="utf-8" ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line isn't needed anymore since v11. Not a blocker.
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
This PR has the |
Congratulations, your PR was merged at 68aef37. Thanks a lot for contributing to OCA. ❤️ |
Standard Migration
depends on