-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[17.0][MIG] sale_automatic_workflow_advance: Migration to 17.0 #3234
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
base: 17.0
Are you sure you want to change the base?
[17.0][MIG] sale_automatic_workflow_advance: Migration to 17.0 #3234
Conversation
fcf0d49 to
a098a6b
Compare
|
/ocabot migration sale_automatic_workflow_advance @antonioburic Thanks for this. FYI, you can embed the depending PR here in order to test it on runboat and makes tests running: https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference%28s%29-to-another-pull-request%28s%29 |
22c103c to
91fc70d
Compare
37a0316 to
091bb0f
Compare
| default=_default_journal, | ||
| check_company=True, | ||
| ) | ||
| company_id = fields.Many2one( |
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.
@antonioburic IMHO, this should go into base module and not here
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.
done, in d90f3f7
|
@antonioburic Could you check tests ? |
|
@antonioburic What's the status of this ? |
091bb0f to
ebc6f88
Compare
4aee32f to
d90f3f7
Compare
|
@rousseldenis rebased and tests resolved, thx for the 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.
Functional review LGTM!
based on #2055
dependency
sale_advance_paymentstill in review in #3156