-
Notifications
You must be signed in to change notification settings - Fork 8.5k
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
Conversation
a387801
to
524393f
Compare
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.
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!
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
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. |
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.
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
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.
I left "switch" because I use "select" a lot =)
Also, in v15, you can click the field name to switch to edit mode
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
524393f
to
1c7d72b
Compare
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.
@auva-odoo : I've made most changes and I'm available to discuss what I didn't understand :)
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
content/applications/sales/point_of_sale/pricing/pricelists.rst
Outdated
Show resolved
Hide resolved
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. |
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.
I left "switch" because I use "select" a lot =)
Also, in v15, you can click the field name to switch to edit mode
1c7d72b
to
f930faa
Compare
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.
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:: |
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.
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.
- Define what the calculation is based on (the :guilabel:`Sales Price`, the | ||
:guilabel:`Cost`, or an :guilabel:`Other Pricelist`.) |
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.
"an Other Pricelist" reads weirdly, maybe like this?
- 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`.) |
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. |
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.
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 :)
task-3005170
f930faa
to
ae96b4f
Compare
@robodoo r+ |
task-3005170 closes #5688 Signed-off-by: Xavier Platteau (xpl) <[email protected]>
@LoredanaLrpz @xpl-odoo staging failed: ci/runbot on 436bdd30820f03714d954d7f07eaeb552e9043c4 (view more at https://runbot.odoo.com/runbot/build/52286015) |
@robodoo retry |
task-3005170 closes #5688 Signed-off-by: Xavier Platteau (xpl) <[email protected]>
@LoredanaLrpz @xpl-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
1 similar comment
@LoredanaLrpz @xpl-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
@LoredanaLrpz @xpl-odoo this pull request has forward-port PRs awaiting action (not merged or closed): |
task-3005170