[IMP] inventory: replace purchase/manufacturing lead times with horizon days#16985
Closed
[IMP] inventory: replace purchase/manufacturing lead times with horizon days#16985
Conversation
Collaborator
|
|
huisit
force-pushed
the
19.0-inventory-replace-security-lead-times-with-horizon-days-rexhu
branch
3 times, most recently
from
c9ab801 to
6145801
Compare
Contributor
Author
|
@theRealThagomizer this is ready for review! |
Contributor
|
due to the complexity of the topic, im upping the scope from 3 to 5 even though it's not a full article refresh (: |
huisit
force-pushed
the
19.0-inventory-replace-security-lead-times-with-horizon-days-rexhu
branch
from
6145801 to
df742aa
Compare
theRealThagomizer
approved these changes
Mar 23, 2026
Contributor
theRealThagomizer
left a comment
theRealThagomizer
left a comment
There was a problem hiding this comment.
Review complete! I think the image files might all need to get compressed; they look a bit big and at least one is too large width-wise. I left some comments throughout about commas, but those can be considered stylistic. Nothing looked like it needed a rewrite, so once you double-check the images and decide which commas to keep and which to remove, this is ready to move on!
...s/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times/all-lead-times.png
Show resolved
Hide resolved
...s/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times/global-example.png
Show resolved
Hide resolved
...entory_and_mrp/inventory/warehouses_storage/replenishment/lead_times/purchase-lead-times.png
Show resolved
Hide resolved
...tions/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times/set-vendor.png
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
huisit
force-pushed
the
19.0-inventory-replace-security-lead-times-with-horizon-days-rexhu
branch
2 times, most recently
from
453925a to
b98dd14
Compare
Contributor
Author
|
Thanks for catching that @theRealThagomizer 🙏 |
huisit
force-pushed
the
19.0-inventory-replace-security-lead-times-with-horizon-days-rexhu
branch
from
b98dd14 to
5b3dd13
Compare
Felicious
reviewed
Mar 24, 2026
Contributor
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Great work with this PR, @huisit !
I can tell you put in a lot of time and effort understanding the nuances of lead times. I pointed out 2 places where I was confused about the lead times functionality. We might have to test it in runbot to confirm. Let's talk about it in person!
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
...ent/applications/inventory_and_mrp/inventory/warehouses_storage/replenishment/lead_times.rst
Outdated
Show resolved
Hide resolved
…on days Co-authored-by: theRealThagomizer <96515928+theRealThagomizer@users.noreply.github.com>
huisit
force-pushed
the
19.0-inventory-replace-security-lead-times-with-horizon-days-rexhu
branch
from
5b3dd13 to
18e2663
Compare
Contributor
|
@robodoo r+ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
documentation task card: https://www.odoo.com/odoo/project.task/6046072
key change:
misc. changes:
types of lead timessection.types of lead timeslist to reflect order of operations in JIT logic.note:
This 19.0 PR can be FWP up to master.