-
-
Notifications
You must be signed in to change notification settings - Fork 534
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
ADD module contract_fixed_discount #1009
ADD module contract_fixed_discount #1009
Conversation
30fa2dc
to
904c588
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
f2bd758
to
41a9d72
Compare
|
||
<field name="arch" type="xml"> | ||
<field name="discount" position="after"> | ||
<field name="discount_fixed" optional="hide" /> |
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.
testing on runboat :
- this looks like to manage the popup form view which which don't care about optional
- so we should add it on the tree view in the contract form view
- we should probably use same group as the discount fields
- I think we should change the group to rely on the same one present on sale order:
groups="product.group_discount_per_so_line"
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.
Test on runboat: thanks ❤️
4f6e6e5
to
9f80be2
Compare
9f80be2
to
d68f30a
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
No description provided.