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
[IMP] *: Convert single create method into multi #80824
Conversation
a460c3c
to
165dc60
Compare
165dc60
to
fcb685c
Compare
45debc6
to
58685fe
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.
wip review
# YTI FIXME: The fix is not at the correct place | ||
# It be set when we actually create the pricelist |
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.
TO fix 👀
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.
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.
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.
Hello @tivisse
Very nice cleaning!
Had a quick look at our modules and left a few comments/suggestions.
Cheers.
9bcda6d
to
40b2654
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.
Thanks for your PR ! Nice technical cleaning :) I did a validation for SM + base.
f9ee6ec
to
08b432e
Compare
TaskID: 2703085 Co-authored-by: Victor Feyens <vfe@odoo.com>
This makes the inversion of fields deterministic, as some orderings make street fields crash in module base_address_extended.
da4378c
to
be7785b
Compare
@robodoo r+ rebase-ff |
Merge method set to rebase and fast-forward |
Staging failed: ci/runbot (view more at http://runbot.odoo.com/runbot/build/11601753) |
@robodoo retry |
Taskid: 2703085 Part-of: #80824 Co-authored-by: Victor Feyens <vfe@odoo.com>
This makes the inversion of fields deterministic, as some orderings make street fields crash in module base_address_extended. closes #80824 Related: odoo/enterprise#22711 Related: odoo/upgrade#3093 Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>
Description of the issue/feature this PR addresses:
Current behavior before PR:
Desired behavior after PR is merged:
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr