[IMP] Inventory: make Storage Categories current with 18.0 #15090
Conversation
|
|
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Good job testing doing this deep dive, @erjer-odoo !
Appreciate the strenuous testing you did and challenging my misunderstandings of the putaway behavior to result in a more accurate document! To make sure your new findings are properly highlighted, I'd like to suggest we move some things around and map out the workflows in even more detail
Let me know if I can clarify any of my comments!
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/shipping_receiving/daily_operations/storage_category.rst
Outdated
Show resolved
Hide resolved
erjer-odoo
force-pushed
the
18.0-Inventory-storage-categories-erjer
branch
from
1d3d322 to
1517ce9
Compare
|
Updated with comments from @Felicious . |
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Great work testing this feature meticulously, @erjer-odoo !
Had super minor things I found and this PR is ready to merge
@robodoo delegate+
| - :guilabel:`Allow mixed products`: several different products can be stored in this location at | ||
| the same time. |
There was a problem hiding this comment.
I ran make review and noticed that this line ends too early. Catching my mistake from 17 months ago 😅
| the same time. | |
| - :guilabel:`Allow mixed products`: several different products can be stored in this location at the | |
| same time. |
| .. image:: storage_category/package-stock-move.png | ||
| :alt: Update the Stock move box to route pallets to the correct locations before validating. | ||
|
|
||
| - If `PAL1` and `PAL2` are full, the pallet is redirected to `WH/Stock/Pallets`. |
There was a problem hiding this comment.
(opinion) the last bullet point gets buried and looks easy to miss below the image. It might be a little bit out of order, but what do you think about listing the partially full bullet last, and moving the "full" bullet point above the image?
|
I'd typically rate this as a 2pt update since it mainly brings the doc from version 18 to 19 with under 50% of the lines changed. That said, given the depth of research and testing you invested (especially around a complex feature that even the functional expert found challenging 🫶 ) I’m increasing the value to 3pts. This reflects the extra effort you put into ensuring accuracy and completeness in the original content! |
Making storage categories documentation current with 18.0, requested here:: https://www.odoo.com/odoo/my-tasks/4681301