-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[16.0][MIG] sale_order_line_price_history: Migration to version 16.0 #2496
[16.0][MIG] sale_order_line_price_history: Migration to version 16.0 #2496
Conversation
App price history of a product from a sale order line and set one of this old prices in the sale order line.
Currently translated at 100.0% (29 of 29 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_line_price_history Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_line_price_history/es/
This way, the user doesn't have to save the line first to see the customer prices. TT29630
Co-authored-by: Ruchir Shukla <[email protected]>
ping @sergio-teruel , @pedrobaeza |
/ocabot migration sale_order_line_price_history |
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.
JS code should be converted to new OWL
7ae0460
to
e945cc4
Compare
Done in sale_order_line_price_history/static/src/js/sale_line_price_history_widget.js |
e945cc4
to
7be32b3
Compare
09e6754
to
ac70f63
Compare
ac70f63
to
e9be72e
Compare
Migrate the SaleOrderLinePriceHistoryWidget to PriceHistoryWidget using the new OWL 16
e9be72e
to
a7664c7
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.
Functional review ok
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.
/ocabot merge nobump
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 116ddfb. Thanks a lot for contributing to OCA. ❤️ |
ping @Juangomezzurita |
@Tecnativa TT42430