-
Notifications
You must be signed in to change notification settings - Fork 10.8k
[IMP] Inventory: Bringing Closest location removal strategy current #15744
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
|
Hi @jero-odoo -- I just updated the "Closest location" removal strategy document to be current with 18.3 forward. Could you review when you get a moment? Thank you! |
jero-odoo
left a comment
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.
Hey @erjer-odoo nice work on this update! Just a few quick changes, then this can go to final/tech. Thanks!
...tions/inventory_and_mrp/inventory/shipping_receiving/removal_strategies/closest_location.rst
Outdated
Show resolved
Hide resolved
...tions/inventory_and_mrp/inventory/shipping_receiving/removal_strategies/closest_location.rst
Outdated
Show resolved
Hide resolved
...tions/inventory_and_mrp/inventory/shipping_receiving/removal_strategies/closest_location.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Jess Rogers <104637850+jero-odoo@users.noreply.github.com>
45c11b4 to
864c4f8
Compare
|
Hi @Felicious ! @jero-odoo has reviewed and approved this PR. Could you take a look when you have a moment and help me merge if you also approve? Thanks! |
Felicious
left a comment
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.
@robodoo r+
closes #15744 Signed-off-by: Felicia Kuan (feku) <feku@odoo.com> Co-authored-by: Jess Rogers <104637850+jero-odoo@users.noreply.github.com>

While working on the removal strategies overview script, I realized that our "Closest Location" removal strategy documentation was slightly out-of-date. In this PR, I did the following:
This required less work than the other removal strategy docs, so I labeled it accordingly.
This saas-18.3 PR can be FWP up to master.