-
Notifications
You must be signed in to change notification settings - Fork 0
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
[14.0][BPRT] edi_sale_ubl_oca #24
base: 14.0
Are you sure you want to change the base?
Conversation
The Form class is just too dumb. If a field is not there it's broken and if you pass a view for a main record you cannot pass any for the x2m fields which makes it impossible to use for general purpose.
Additionally, avoid recomputing states for all lines when we know which lines have changed.
Several lines' state make the order not satisfied.
146a63b
to
7747214
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.
Hi a @thienvh332 , I have put some reviews, could you double-check again?
9813882
to
1081bb3
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.
1081bb3
to
5e6d659
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.
[IMP] edi_sale_ubl_oca: depend on edi_exchange_type_auto
429ef55
to
50ed1be
Compare
50ed1be
to
0f78b1f
Compare
No description provided.