Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[IMP] inventory: update 3 step receipts doc #8838

Closed
wants to merge 1 commit into from
Closed

[IMP] inventory: update 3 step receipts doc #8838

wants to merge 1 commit into from

Conversation

brse-odoo
Copy link
Contributor

@brse-odoo brse-odoo commented Apr 18, 2024 •
edited

This PR is an [IMP] to the receipts_three_steps doc in Odoo's Inventory documentation, specific to 17.0.

Here's what I did:

  • Retooled some of the instructions/flow for 17.2 update that affected all multi-step routes
  • Replaced all screenshots for 17.2 UI, removed some screenshots no longer relevant

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

@brse-odoo brse-odoo self-assigned this Apr 18, 2024
@robodoo
Copy link
Collaborator

robodoo commented Apr 18, 2024

Pull request status dashboard.

@C3POdoo C3POdoo requested a review from a team April 18, 2024 14:43
@brse-odoo
Copy link
Contributor Author

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!

@brse-odoo brse-odoo changed the title [IMP] update 3 step receipts doc [IMP] inventory: update 3 step receipts doc Apr 18, 2024
hojo-odoo
hojo-odoo approved these changes Apr 19, 2024
View reviewed changes
Copy link
Contributor

@hojo-odoo hojo-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @brse-odoo,

Nice job on this update. I just had one small suggestion.

Thanks,
hojo

...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Outdated Show resolved Hide resolved
@brse-odoo brse-odoo force-pushed the 17.0-inventory-update-3-step-receipts-brse branch from 748f004 to db4a102 Compare April 22, 2024 13:52
@brse-odoo
Copy link
Contributor Author

Hi @odoo/us-doc-review, this PR is ready for final content review. Thank you!

@brse-odoo brse-odoo requested a review from a team April 22, 2024 15:43
StraubCreative
StraubCreative approved these changes Apr 22, 2024
View reviewed changes
Copy link
Contributor

@StraubCreative StraubCreative left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @brse-odoo
Approving with comments.

Please address the zoomed-in screenshots:

  • receipts-three-steps-internal-transfer.png
  • receipts-three-steps-receipt-form.png
  • receipts-three-steps-second-transfer.png
  • receipts-three-steps-smart-button.png

...as well as the few sections that repetitively begin with Click.

Then can pass to SALI, thank you!

...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Show resolved Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Show resolved Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Outdated Show resolved Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Show resolved Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/daily_operations/receipts_three_steps.rst Outdated Show resolved Hide resolved
@brse-odoo brse-odoo force-pushed the 17.0-inventory-update-3-step-receipts-brse branch from db4a102 to 1c0702d Compare April 22, 2024 19:56
@brse-odoo
Copy link
Contributor Author

Hi @brse-odoo Approving with comments.

Please address the zoomed-in screenshots:

  • receipts-three-steps-internal-transfer.png
  • receipts-three-steps-receipt-form.png
  • receipts-three-steps-second-transfer.png
  • receipts-three-steps-smart-button.png

...as well as the few sections that repetitively begin with Click.

Then can pass to SALI, thank you!

Hi @samueljlieber this PR has passed us-doc-review and is ready for your lookover before merge, thank you!

@brse-odoo brse-odoo requested a review from a team April 22, 2024 20:01
samueljlieber
samueljlieber approved these changes Apr 23, 2024
View reviewed changes
Copy link
Contributor

@samueljlieber samueljlieber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @brse-odoo, nice PR! Your changes look good to me 👍
..
@robodoo r+

@robodoo robodoo closed this in ba2a95a Apr 24, 2024
@fw-bot fw-bot mentioned this pull request Apr 24, 2024
Closed
This was referenced Apr 24, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StraubCreative StraubCreative StraubCreative approved these changes

@samueljlieber samueljlieber samueljlieber approved these changes

@hojo-odoo hojo-odoo hojo-odoo approved these changes

Assignees

@brse-odoo brse-odoo

Labels
2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@brse-odoo @robodoo @StraubCreative @samueljlieber @hojo-odoo