-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[IMP] inventory: update 3 step delivery doc #8837
Conversation
Hi @odoo/inventory-doc-review, this PR is ready for review when one of you has a chance. All that needs to be reviewed are the few lines that were changed, and the replaced screenshots. Thank you! |
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 looks great, @brse-odoo ! I just had super minor suggestions for you to implement as you see fit. 😄
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Show resolved
Hide resolved
cdcbb76
to
8279ea5
Compare
Hi @odoo/us-doc-review, this PR is ready for final content review. Thank you! |
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.
Hi @brse-odoo
I like the revisions you made in the language and on a few screenshots.
For a few screenshot changes though, I'd suggest either reverting back to the former screenshot or retaking to cover more of the UI. I wrote my logic as to why I think this on the comments.
After reviewing/revising, please tag SALI for merge, thanks!
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Show resolved
Hide resolved
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.
Another item, see below please @brse-odoo
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/delivery_three_steps.rst
Show resolved
Hide resolved
8279ea5
to
0b073aa
Compare
Hi @samueljlieber, this PR has passed us-doc-review and is ready for your final lookover before merge, thank you! |
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.
Hi @brse-odoo, the changes in your PR looks good to me 👍 Nice work, thank you!
..
@robodoo r+
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
9 similar comments
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
@brse-odoo @samueljlieber this pull request has forward-port PRs awaiting action (not merged or closed): |
This PR is an [IMP] to the
delivery_three_steps
doc in Odoo's Inventory documentation, specific to 17.0.Here's what I did:
Note: This PR was only to update small instructions and replace screenshots. A second, targeted PR for a more robust rewrite will follow once this is merged.
Project Task Link