-
-
Notifications
You must be signed in to change notification settings - Fork 759
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] account_sequence_option: Migration to 17.0 #1921
base: 17.0
Are you sure you want to change the base?
Conversation
Refer to account_sequence with a fully qualified reference. Convert fully qualified references to the example sequences to relative references. This allows the demo xml file to be used as a template for a custom module's data xml file
…s when returning to draft and changing date.
Currently translated at 87.5% (7 of 8 strings) Translation: account-financial-tools-14.0/account-financial-tools-14.0-account_sequence_option Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-14-0/account-financial-tools-14-0-account_sequence_option/it/
Currently translated at 87.5% (7 of 8 strings) Translation: account-financial-tools-14.0/account-financial-tools-14.0-account_sequence_option Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-14-0/account-financial-tools-14-0-account_sequence_option/it/
Currently translated at 100.0% (8 of 8 strings) Translation: account-financial-tools-14.0/account-financial-tools-14.0-account_sequence_option Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-14-0/account-financial-tools-14-0-account_sequence_option/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_sequence_option Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_sequence_option/
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, tested in runboat
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
@JordiBForgeFlow can you review please. Thanks! |
Please can you rename the migration commit? It's missing the module name |
1ba59e9
to
77f08e6
Compare
Done, thanks! |
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!
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 Test. it's work 👍
but I don't know what is commit for? [DON'T MERGE] test-requirements.txt
Module migrated to version 17.0
cc https://github.com/APSL 159235
@miquelalzanillas @lbarry-apsl @javierobcn @mpascuall @BernatObrador @ppyczko please review
[Issue] Migration to version 17.0 account-analytic#607
Depends on:
[base_sequence_option] [17.0][MIG] base_sequence_option: Migration to 17.0 server-tools#3009