-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[17.0][IMP]sale_quotation_number: Boolean to determine whether quotation sequence has been used #3534
[17.0][IMP]sale_quotation_number: Boolean to determine whether quotation sequence has been used #3534
Conversation
1130c42
to
57f5f82
Compare
FW port not neccesary because it has already been fixed.
|
4d7c2a4
to
9f76d8b
Compare
9f76d8b
to
94e7bba
Compare
…quence has been used
94e7bba
to
af16f0c
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 👍🏻
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.
Technical review LGTM
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
/ocabot merge patch |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at ad03b66. Thanks a lot for contributing to OCA. ❤️ |
FW Port of #2719
The current fix doesn't work because it stops working if you use formulas in the prefixes of the succesors.
T-7243