-
-
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
[15.0][MIG] sale_order_revision #2381
Conversation
…module by Agile Business Group
* extract description from __openerp__.py * move things according to standard OCA layout * reformatting * port to new API * Major rewrite of the 7.0 version: * the name of the SO is kept, and a revision number is appended * old revisions are made inactive * rewrite the copy logic * update translations, added en.po fr.po * fix typo in README * use post_init_hook * more assertions in the test * sale_order_revision: remove en.po * current_revision is only visible for non current revisions
* Fix create revision and improve view invisibility * Fix unrevisioned name and improve test
wrong keyword name in SaleOrder.copy definition was breaking sale order duplication
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/
Currently translated at 100.0% (14 of 14 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/de/
Currently translated at 100.0% (14 of 14 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/zh_CN/
Currently translated at 100.0% (14 of 14 strings) Translation: sale-workflow-12.0/sale-workflow-12.0-sale_order_revision Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-12-0/sale-workflow-12-0-sale_order_revision/es/
[13.0] fix pre-commit + formatting add test lines and linting xml
[UPD] README.rst
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: sale-workflow-13.0/sale-workflow-13.0-sale_order_revision Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_order_revision/
[UPD] README.rst
…rks as expected. TT33433
Hi @OriolVForgeFlow, |
/ocabot migration sale_order_revision |
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 review
@pedrobaeza can you merge this? |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
It looks like something changed on |
Congratulations, your PR was merged at 7cd3a76. Thanks a lot for contributing to OCA. ❤️ |
Followup of #2275
@simahawk @grindtildeath @cyrilmanuel @leemannd