-
-
Notifications
You must be signed in to change notification settings - Fork 788
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][REF] migrate and refactor purchase_analytic_global #801
[12.0][REF] migrate and refactor purchase_analytic_global #801
Conversation
An analytic account field on the purchase order that applies the account on all the lines of the purchase order.
0cbff74
to
ce837b6
Compare
5c6008b
to
f4409b6
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
Small proposal to improve the help string.
8090bba
to
50b8867
Compare
@yvaucher updated, squashed |
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-801-by-dreispt-bump-no. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
|
||
account_analytic_id = fields.Many2one( | ||
"account.analytic.account", | ||
string="Analytic Account", |
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.
Odoo doesn't like same string in 2 fields, you must rename it, because Analytic Account it is already used on account_analytic_ids field.
Congratulations, your PR was merged at c237c9a. Thanks a lot for contributing to OCA. ❤️ |
migrate #228