-
-
Notifications
You must be signed in to change notification settings - Fork 701
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
[17.0][MIG] product_form_account_move_line_link: Migration to 17.0 #1676
[17.0][MIG] product_form_account_move_line_link: Migration to 17.0 #1676
Conversation
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.
code+usage LGTM
ea807d2
to
5b79f80
Compare
@JaumeBforgeFlow would you be so kind and make the second review of this migration? |
913b136
to
a802b55
Compare
a802b55
to
159918f
Compare
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.
LGTM as well, technical review
@pedrobaeza could you merge? |
/ocabot migration product_form_account_move_line_link |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at e09f8b5. Thanks a lot for contributing to OCA. ❤️ |
No description provided.