-
-
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
[12.0][BACKPORT] account_move_post_date_user #1971
[12.0][BACKPORT] account_move_post_date_user #1971
Conversation
Currently translated at 100.0% (3 of 3 strings) Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_post_date_user Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_post_date_user/it/
Currently translated at 100.0% (3 of 3 strings) Translation: account-financial-tools-16.0/account-financial-tools-16.0-account_move_post_date_user Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_move_post_date_user/it/
Backport module from OCA#1822 In order to adapt it to v13.0, the `_post` method has been renamed to `post` and adapting the logic. Also, the importing on tests has been adapted.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-financial-tools-13.0/account-financial-tools-13.0-account_move_post_date_user Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-13-0/account-financial-tools-13-0-account_move_post_date_user/
9987773
to
eee6a7c
Compare
eee6a7c
to
ba99cc2
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.
code review 👍🏼
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! functional and code review 👍🏿
A minor comment, perhaps the commit message should be BKP or something that does not relate to a migration?
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.
ty
ba99cc2
to
e1157ac
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!
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 344452f. Thanks a lot for contributing to OCA. ❤️ |
Backport of account_move_post_date_user from 13.0
cc: @JordiMForgeFlow