-
-
Notifications
You must be signed in to change notification settings - Fork 750
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 purchase_price_recalculation module #725
Conversation
8ea3eb5
to
676de50
Compare
please @pedrobaeza |
Hey, you abuse this wildcard, hehe. I will try to check later. |
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.
One minor change and one question.
Otherwise, LGTM. Usefull module. Thanks a lot @bealdav
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.
Putting as server action allows you to recompute prices on already locked or confirmed purchase orders, which is not desirable. You can prevent that showing any error on that cases.
3ff32e7
to
c9e8c8e
Compare
c9e8c8e
to
70163d8
Compare
@pedro fixed as you would |
Please @pedrobaeza could you have a look here |
Please @pedrobaeza could u merge this one. Thanks a lot |
LGTM (code review) |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 3db487e. Thanks a lot for contributing to OCA. ❤️ |
No description provided.