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.

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] Sales - Update optional_products.rst #4404

Closed
wants to merge 1 commit into from

Conversation

ksc-odoo
Copy link
Contributor

@ksc-odoo ksc-odoo commented May 10, 2023

adhered new formatting rules to old doc (v14) related to Optional Products (under Sales > Send Quotations) @samueljlieber

Project Task: 3329153

@ksc-odoo ksc-odoo self-assigned this May 10, 2023
@robodoo
Copy link
Collaborator

robodoo commented May 10, 2023

@C3POdoo C3POdoo requested a review from a team May 10, 2023 20:55
Copy link
Contributor Author

@ksc-odoo ksc-odoo left a comment

Choose a reason for hiding this comment

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

👍

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 @ksc-odoo I have a handful of technical changes to this PR, please see them below. Let me know if you have any questions or want help with any of the edits! Thank you :)

Comment on lines 6 to 7
along with a core product. The aim is to offer useful and related products to customers, which may result
in an increased sale.
Copy link
Contributor

Choose a reason for hiding this comment

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

Late 100th char line break

(under the **Optional Products** tab, as you can see below).
With the Odoo *Sales* application, it is possible to add or modify optional products directly on
quotations (located under the :guilabel:`Optional Products` tab on a quotation form, in the *Sales*
application).

.. image:: optional_products/optional_products_2.png
Copy link
Contributor

Choose a reason for hiding this comment

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

Note on images - since we are already improving this document, please update the images to follow our guidelines being sure to retaking screenshots if they cannot be resized and still be legible as well as have the correct filename convention (maximum width of 933px or ideally 768px wide & descriptive hyphenated filename). Please let me know if you have any questions! 🙂


.. image:: optional_products/optional_products_2.png
:align: center
:class: img-thumbnail
:alt: How to add optional products to your quotations on Odoo Sales
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add periods to alt tags

using associated carts, with a user-friendly layout. More than that, if a customer selects all the
optional products suggested, these additional items will automatically fill in the quotation
managed by the salesman.
In practice, your customers are able to add different optional products to an order by clicking the
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
In practice, your customers are able to add different optional products to an order by clicking the
In practice, customers are able to add different optional products to an order by clicking the


Be sure to check out our documentation about :doc:`quote_template` to understand how you can
enable, create, design and manage your own quotation templates before reading this part.
Be sure to review the documentation about :doc:`quote_template` to better understand how quotation
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use global relative paths rather than local relative paths

Suggested change
Be sure to review the documentation about :doc:`quote_template` to better understand how quotation
Be sure to review the documentation about
:doc:`/applications/sales/sales/send_quotations/quote_template` to better understand how quotation

@ksc-odoo
Copy link
Contributor Author

Hi @ksc-odoo I have a handful of technical changes to this PR, please see them below. Let me know if you have any questions or want help with any of the edits! Thank you :)

Hey @samueljlieber thanks for the awesome technical review. I made all the suggested changes (both RST and image-wise), which can be found here: https://drive.google.com/drive/folders/1OtwEbMmzukSVRjGCWMzt8YadC-OiQ6Vc?usp=sharing

If there's anything else you need from me, just let me know. thanks again! 👍

@ksc-odoo ksc-odoo force-pushed the 14.0-sales-update-optional-products-ksc branch from 1fff1e9 to 5a25828 Compare May 17, 2023 22:09
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 @ksc-odoo nice job with making the previous changes, I caught a couple more, please see below. Once these changes are made, tag me once more and this PR should be good to go for final review. Thank you! 🙂

Comment on lines 59 to 83
products added under the :guilabel:`Optional Products` tab are present in the quotation, by
default, whenever that particular quotation template is used.
Copy link
Contributor

Choose a reason for hiding this comment

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

early 100th char line break

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see. I only made this break because I thought that since the comma was right after the word "default" (in other words no spaces separating the two) that it needed to be knocked down to the next line. No worries, though. I slightly restructured the sentence to make everything work. Thanks 👍


.. seealso::
- :doc:`quote_template`
:doc:`quote_template`
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use global relative paths instead of local relative paths

@ksc-odoo ksc-odoo force-pushed the 14.0-sales-update-optional-products-ksc branch from 5a25828 to bd49689 Compare May 18, 2023 17:20
@ksc-odoo
Copy link
Contributor Author

Hi @ksc-odoo nice job with making the previous changes, I caught a couple more, please see below. Once these changes are made, tag me once more and this PR should be good to go for final review. Thank you! 🙂

Alrighty @samueljlieber - i made all the necessary changes - thanks again for your consistently AWESOME reviews/feedback/help/expertise! Let me know if there's anything else I need to adjust before it can move on to the Final Review stage - thanks again! 👍

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 @ksc-odoo great job with this PR - looks good to me! Moving to final review @StraubCreative 🙂

@StraubCreative StraubCreative force-pushed the 14.0-sales-update-optional-products-ksc branch from bd49689 to f37cd95 Compare June 15, 2023 14:18
@StraubCreative
Copy link
Contributor

Fresh rebase on f37cd95.

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.

Hello @ksc-odoo 👋
Couple change requests below, thanks!

@ksc-odoo ksc-odoo force-pushed the 14.0-sales-update-optional-products-ksc branch 2 times, most recently from 89dbc81 to cde7787 Compare June 15, 2023 18:19
@ksc-odoo
Copy link
Contributor Author

Hello @ksc-odoo 👋 Couple change requests below, thanks!

Thanks for the awesome suggestions! Ready for another quick look, whenever you get a chance 👍 @StraubCreative

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 @ksc-odoo
approving with 2 comments
merge when ready 👍

@robodoo delegate=ksc-odoo

@ksc-odoo ksc-odoo force-pushed the 14.0-sales-update-optional-products-ksc branch from cde7787 to 9e7d32f Compare August 8, 2023 17:50
@ksc-odoo
Copy link
Contributor Author

ksc-odoo commented Aug 8, 2023

@robodoo r+

robodoo pushed a commit that referenced this pull request Aug 8, 2023
closes #4404

Signed-off-by: Kevin Scannell (ksc) <ksc@odoo.com>
@robodoo robodoo temporarily deployed to merge August 8, 2023 19:09 Inactive
@robodoo robodoo closed this Aug 8, 2023
4 similar comments
@fw-bot
Copy link
Collaborator

fw-bot commented Aug 17, 2023

@ksc-odoo this pull request has forward-port PRs awaiting action (not merged or closed):

1 similar comment
@fw-bot
Copy link
Collaborator

fw-bot commented Aug 18, 2023

@ksc-odoo this pull request has forward-port PRs awaiting action (not merged or closed):

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

Successfully merging this pull request may close these issues.

5 participants