-
Notifications
You must be signed in to change notification settings - Fork 10.2k
[IMP] Purchase: update control policy doc #14566
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
Conversation
d9626a2
to
a17da43
Compare
a17da43
to
28e548f
Compare
@larm-odoo this is ready for review. 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.
Hi @jero-odoo -straightforward and concise. Awesome doc- I just had a couple of comments. Approved!
content/applications/inventory_and_mrp/purchase/manage_deals/control_bills.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/purchase/manage_deals/control_bills.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/purchase/manage_deals/control_bills.rst
Show resolved
Hide resolved
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.
Just wanted to complete this review early! it looks good to me, @jero-odoo (:
@robodoo delegate+
content/applications/inventory_and_mrp/purchase/manage_deals/control_bills.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/purchase/manage_deals/control_bills.rst
Show resolved
Hide resolved
28e548f
to
187ca14
Compare
@robodoo r+ |
closes #14566 Signed-off-by: Jessica Rogers (jero) <jero@odoo.com>
Docs task: https://www.odoo.com/mail/view?model=project.task&res_id=4984465&access_token=7425c8c2-7341-4274-a47d-33b7d284e154
Since 18.2 there is no more control policy setting anymore.
Now the default value depends on product type:
Service = based on ordered
Goods = based on delivered