[IMP] Inventory: Update lot numbers documentation #15732
Conversation
|
|
|
Hi @theRealThagomizer ! Continuing last week's work on removal strategies documentation, I realized I also needed to update the "Lot numbers" documentation to be current. When you get a moment, can you review this and let me know what you think? Thank you! |
Reviewing now! Sorry this slipped past me while I was out of town. |
theRealThagomizer
left a comment
theRealThagomizer
left a comment
There was a problem hiding this comment.
Hiya, @erjer-odoo! The material you updated looks good, but I think it might be worth tweaking the info hierarchy a little bit here. I got kind of confused by the actual layout of the article as I was looking at it on Odoo documentation, and I feel like the way the article is broken down works broadly, but less so under the "Assign lots for shipping and receiving" header. Maybe give it a quick speedread and see if you think it would benefit from reorganizing the info some? I left a suggestion in the comment.
Marking as approved since there's nothing here that needs a major reworking!
content/applications/inventory_and_mrp/inventory/product_management/product_tracking/lots.rst
Outdated
Show resolved
Hide resolved
content/applications/inventory_and_mrp/inventory/product_management/product_tracking/lots.rst
Outdated
Show resolved
Hide resolved
erjer-odoo
force-pushed
the
saas-18.3-inventory-lot-numbers-erjer
branch
from
7d2feed to
4bf336c
Compare
|
Hi @Felicious ! @theRealThagomizer has approved this PR. When you have a moment, could you take a look? If you also approve, could you help me merge? Thank you! |
closes #15732 Signed-off-by: Felicia Kuan (feku) <feku@odoo.com>
4 participants
While updating documentation for removal strategies documentation, I realized that the "Lot numbers" document is out of date. I took this opportunity to update to make it current from 18.3 forward. Namely, the list icon where we used to specify lot/serial numbers has changed to a "Details" link, and the name of the window that opens when you click it is now called "Detailed Operations." I've updated the document to be current with this information.
This saas-18.3 PR can be FWP up to master.