Skip to content

[IMP] inventory: update one step shipments doc #9120

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

Closed
wants to merge 1 commit into from

Conversation

brse-odoo
Copy link
Contributor

@brse-odoo brse-odoo commented May 2, 2024

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

Here's what I did:

  • Rewrote/updated copy and sections for accuracy specific to 17.0
  • Replaced screenshots/added new screenshots with 17.0 UI
  • Added admonitions and expanded doc to be more robust

Project Task Link

@brse-odoo brse-odoo self-assigned this May 2, 2024
@robodoo
Copy link
Collaborator

robodoo commented May 2, 2024

@C3POdoo C3POdoo requested a review from a team May 2, 2024 14:59
@brse-odoo
Copy link
Contributor Author

@larm-odoo This PR is ready for content review, thank you in advance!

@brse-odoo brse-odoo requested a review from larm-odoo May 2, 2024 15:17
Copy link
Contributor

@larm-odoo larm-odoo left a comment

Choose a reason for hiding this comment

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

Hi Brandon- nice doc, easy and understandable. I don't have a lot of suggestions- just two, really. I am approving now!

@brse-odoo brse-odoo force-pushed the 17.0-inventory-update-one-step-brse branch from f1079ef to 79902e7 Compare May 2, 2024 18:48
@brse-odoo
Copy link
Contributor Author

@odoo/us-doc-review This PR is ready for final content review, thank you!

@brse-odoo brse-odoo requested a review from a team May 2, 2024 18:49
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, everything looks good– however, please resolve the build errors before I approve? Thank you 🙏

@brse-odoo brse-odoo force-pushed the 17.0-inventory-update-one-step-brse branch from 79902e7 to 1081057 Compare May 6, 2024 13:31
@brse-odoo
Copy link
Contributor Author

Hi @brse-odoo, everything looks good– however, please resolve the build errors before I approve? Thank you 🙏

Hey @samueljlieber, sorry about that - all good to go now 👍

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.

Thank you @brse-odoo, your improvements in this PR look good to me! Nice work 👍
...
@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants