-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[REF] inventory: move removal strats into own folder #7530
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ The *First Expired, First Out* (FEFO) removal strategy targets products for remo | |
| assigned removal dates. | ||
|
|
||
| .. seealso:: | ||
| :doc:`Other removal strategies <removal>` | ||
| :doc:`About removal strategies <../removal_strategies>` | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same here as my other comment |
||
|
|
||
| .. _inventory/warehouses_storage/removal-date: | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ The *Least Packages* removal strategy fulfills an order by opening the fewest nu | |
| which is ideal for maintaining organized stock without needing to open multiple boxes. | ||
|
|
||
| .. seealso:: | ||
| - :doc:`Other removal strategies <removal>` | ||
| - :doc:`About removal strategies <../removal_strategies>` | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Technically, changing the wording would be part of another commit since it could also be applied to older versions than 17.0. But this is such a minor change that you can disregard my comment. It's mostly to keep in mind that when we apply different types of changes, they may apply to different versions. |
||
| - `Odoo Tutorials: Least Packages <https://www.odoo.com/slides/slide/5477/share>`_ | ||
|
|
||
| To understand how the removal strategy works, consider the following example, featuring a warehouse | ||
|
|
||
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.
This is out of scope of this PR, but want to note that this table does not format well on mobile. Maybe in another [IMP] PR this could be adjusted? I could see a card layout being useful here, the Accounting and Invoicing doc is a good example.