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] pos: update content of the pricelists page #5688

Closed
wants to merge 1 commit into from

Conversation

LoredanaLrpz
Copy link
Contributor

task-3005170

@robodoo
Copy link
Collaborator

robodoo commented Aug 30, 2023

@LoredanaLrpz LoredanaLrpz force-pushed the 15.0-pos-pricelists-lrpz branch 2 times, most recently from a387801 to 524393f Compare September 4, 2023 07:53
@LoredanaLrpz LoredanaLrpz marked this pull request as ready for review September 4, 2023 07:57
@C3POdoo C3POdoo requested review from a team September 4, 2023 07:58
Copy link
Contributor

@auva-odoo auva-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 @LoredanaLrpz Thank you for writing this, you did a great job :-) I have a few suggestions and a few questions (sorry!), let me know when you're ready to discuss them!

Comment on lines 113 to 116
You can also set a pricelist to be selected automatically once a specific :ref:`customer is set
<pos/customers>`. To do so, go to the customer form and switch to the preferred pricelist on the
:guilabel:`Pricelist` field of the :guilabel:`Sales & Purchase` tab.
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
You can also set a pricelist to be selected automatically once a specific :ref:`customer is set
<pos/customers>`. To do so, go to the customer form and switch to the preferred pricelist on the
:guilabel:`Pricelist` field of the :guilabel:`Sales & Purchase` tab.
You can also set a pricelist to be selected automatically once a specific :ref:`customer is set
<pos/customers>`. To do so, go to the customer form and select the preferred pricelist in the
:guilabel:`Pricelist` field of the :guilabel:`Sales & Purchase` tab.

I would maybe also mention that you need to click Edit, but that's probably me

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 left "switch" because I use "select" a lot =)
Also, in v15, you can click the field name to switch to edit mode

Copy link
Contributor Author

@LoredanaLrpz LoredanaLrpz left a comment

Choose a reason for hiding this comment

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

@auva-odoo : I've made most changes and I'm available to discuss what I didn't understand :)

Comment on lines 113 to 116
You can also set a pricelist to be selected automatically once a specific :ref:`customer is set
<pos/customers>`. To do so, go to the customer form and switch to the preferred pricelist on the
:guilabel:`Pricelist` field of the :guilabel:`Sales & Purchase` tab.
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 left "switch" because I use "select" a lot =)
Also, in v15, you can click the field name to switch to edit mode

@auva-odoo auva-odoo requested a review from a team September 12, 2023 11:43
Copy link
Contributor

@jcs-odoo jcs-odoo left a comment

Choose a reason for hiding this comment

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

Hello @LoredanaLrpz and @auva-odoo

Superrrr great work from both of you! Clean PR and review👌Thanks!

I wanted to address my comments immediately but I don't have the time right now. Maybe on Monday. But anyway:

@robodoo delegate+

Cheers :)

Click :guilabel:`Create` or select an existing pricelist. The pricelist setup form differs depending
on the chosen type of pricelist.

.. tabs::
Copy link
Contributor

Choose a reason for hiding this comment

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

It certainly is!
But it reminds me that tabs' titles are still not translatable, sadly.

There is an issue about this: executablebooks/sphinx-tabs#137
but it seems stalled.

Comment on lines 70 to 71
- Define what the calculation is based on (the :guilabel:`Sales Price`, the
:guilabel:`Cost`, or an :guilabel:`Other Pricelist`.)
Copy link
Contributor

Choose a reason for hiding this comment

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

"an Other Pricelist" reads weirdly, maybe like this?

Suggested change
- Define what the calculation is based on (the :guilabel:`Sales Price`, the
:guilabel:`Cost`, or an :guilabel:`Other Pricelist`.)
- Define what the calculation is based on (:guilabel:`Sales Price`,
:guilabel:`Cost`, or :guilabel:`Other Pricelist`.)

Comment on lines 17 to 19
Then, select the :ref:`type of pricelist <pricelists/create>` by checking :guilabel:`Multiple prices
per product` or :guilabel:`Advanced price rules (discounts, formulas)`, save, and click
:guilabel:`Pricelists` to create new ones or modify existing ones.
Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with @auva-odoo 's comment to better explain the difference between the two types (multiple / advanced rules), or maybe add an example section?

I wanted to do it but I'm afraid I have to leave, so we'll see on Monday morning :)

@xpl-odoo
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Oct 12, 2023
task-3005170

closes #5688

Signed-off-by: Xavier Platteau (xpl) <[email protected]>
@robodoo
Copy link
Collaborator

robodoo commented Oct 12, 2023

@LoredanaLrpz @xpl-odoo staging failed: ci/runbot on 436bdd30820f03714d954d7f07eaeb552e9043c4 (view more at https://runbot.odoo.com/runbot/build/52286015)

@xpl-odoo
Copy link
Contributor

@robodoo retry

@odoo odoo deleted a comment from robodoo Oct 12, 2023
robodoo pushed a commit that referenced this pull request Oct 12, 2023
task-3005170

closes #5688

Signed-off-by: Xavier Platteau (xpl) <[email protected]>
@robodoo robodoo temporarily deployed to merge October 12, 2023 12:59 Inactive
@robodoo robodoo closed this Oct 12, 2023
@fw-bot
Copy link
Collaborator

fw-bot commented Oct 16, 2023

@LoredanaLrpz @xpl-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 Oct 17, 2023

@LoredanaLrpz @xpl-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.

6 participants