[IMP] Inventory: making targeted edits for 16 #4381
Conversation
ksc-odoo
left a comment
ksc-odoo
left a comment
There was a problem hiding this comment.
@larm-odoo just finished my review on this doc - awesome job! Remember to ensure that all headings don't start with an action/verb (per JCS' recent comments) - and refrain from using "+" or "&" in headings, either (per ZST's request). If you have any questions or need clarification on anything, don't hesitate to reach out. Feel free to re-tag me once these changes have been made, so we can make sure everything is good-to-go before moving to Tech Review. Thanks!
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
larm-odoo
force-pushed
the
16.0-inventory-make-targeted-edits-larm
branch
from
cddd67a to
12fd044
Compare
|
Ready for another peek, @ksc-odoo - and thank you! |
|
Thanks again @ksc-odoo! Looks like @StraubCreative is up next |
StraubCreative
requested review from
samueljlieber
and removed request for
StraubCreative
|
@samueljlieber same here— can you take a look first? Ty! |
samueljlieber
left a comment
samueljlieber
left a comment
There was a problem hiding this comment.
Hi @larm-odoo just a couple small technical changes, please see below and then tag me once more after the changes have been made :)
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
...cations/inventory_and_mrp/inventory/management/shipments_deliveries/receipts_three_steps.rst
Show resolved
Hide resolved
larm-odoo
force-pushed
the
16.0-inventory-make-targeted-edits-larm
branch
from
12fd044 to
52d2d3a
Compare
|
hi @samueljlieber - these are all set for you to look at again! |
samueljlieber
left a comment
samueljlieber
left a comment
There was a problem hiding this comment.
Hi @larm-odoo these changes in this PR look good to me, nice job! Passing to @StraubCreative for final review 🙂
samueljlieber
force-pushed
the
16.0-inventory-make-targeted-edits-larm
branch
from
52d2d3a to
25b2c8e
Compare
|
Rebased to latest in 25b2c8e since this PR has not been rebased after the 16.0 Inventory restructure. @larm-odoo and @Felicious is this PR still relevant? |
Yes- I checked the current v16 doc, and it could use this update still. Even with the restructuring of the inventory docs, this information is updated for 16 and is good to go! |
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst
Outdated
Show resolved
Hide resolved
samueljlieber
force-pushed
the
16.0-inventory-make-targeted-edits-larm
branch
from
25b2c8e to
4c61700
Compare
samueljlieber
left a comment
samueljlieber
left a comment
There was a problem hiding this comment.
@StraubCreative this PR is good to go 👍
|
@robodoo r+ |
closes #4381 Signed-off-by: Zachary Straub (zst) <[email protected]>
Updated this doc for specific v 16 changes - no edit button or save button, and to make something it's a 'new' button not 'create'