[IMP] ecommerce/prices: update prices page

[lman-odoo]

task-3711225 and task-3339294

[robodoo]

[StraubCreative]

Sales portion LGTM 👍
Good update!

[auva-odoo]

Thank you for your work @lman-odoo! This is a complex topic, and my review ended up a bit dense—hopefully it’s not too overwhelming 🙈. Here are my main suggestions:

• Taxes: I’d move the Taxes section to the top of the page (see my comment on line 238). It’s a key topic and not just relevant for B2B vs. B2C.
• Discounts: I think we should add a dedicated Discounts section. It should cover all the ways discounts can be configured in eCommerce, with links/refs where relevant: Discounts & Loyalty (link to the Sales docs), Permanent discounts, Pricelists with discounts (link to the Sales docs + pricelist section here on this page), Pricelists with eCommerce promotional codes. And ideally, explain how these interact (this may require some thorough testing).
• eWallets & Gift Cards: These should be explicitly mentioned somewhere in the doc (placement to be decided once you've implemented the other changes.
• Relative paths: When pointing back to /applications, you don’t need to use relative paths and can start from /applications directly. I added an example on line 17, but there are a few other links to update.
• B2B documentation: I think we should create a dedicated B2B page. This topic covers more than just prices (as you note in the doc). Once that’s in place, I’d move the B2B access management section out of the Ordering and Checkout page and into the new page. It’ll be more helpful to have everything centralized. (See also my comment on line 205.) Some useful videos on the topic: https://www.youtube.com/watch?v=OHaRvw0ewnY and https://www.youtube.com/watch?v=mbofJDYnwaw

Please reach out if you have any questions or if anything's unclear! Thank you 🙂

[auva-odoo]

Pricelists can be used for reasons other than having B2B/B2C prices (e.g., condition (e.g., time period, currency, website, category, etc.). That being said, and given the variety of things pricelists are used for, I don't think we need to mention them all here; they can be mentioned in the Pricelists section instead (see my comment on line 65-69.

Suggested change

	Odoo offers multiple options to configure prices on the eCommerce. It is possible to use diffrent
	:ref:`prices per unit <ecommerce/prices/price-per-unit>` and :ref:`discounts
	<ecommerce/prices/permanent-discounts>` and to set up :ref:`pricelists <ecommerce/pricelists>`
	to apply :ref:`B2B/B2C prices <ecommerce/prices/b2b-b2c-pricelists>` and
	:ref:`GeoIP-based <ecommerce/prices/geoip>` prices.
	Odoo offers multiple options to configure product prices in eCommerce. For example, you can apply
	:ref:`prices per unit <ecommerce/prices/price-per-unit>`, offer :ref:`permanent discounts
	<ecommerce/prices/permanent-discounts>`, and set up :ref:`pricelists <ecommerce/pricelists>`.

[auva-odoo]

I think this could go in an example admonition, and I'd modify it a bit as follows:

• I'd add an intro sentence explaining the use case on the screenshots very briefly (i.e;, a 2L can of paint sold for 60$, i.e., 30$/L)
• I would change the first screenshot so we actually see 1) the selected unit (Base Unit count 2nd field, i.e., "L") and 2) the sales price per Units as well

(we could probably hide the fields that are not relevant for this use case)

[auva-odoo]

• I think we should go a bit deeper into the fields on the product form (in particular, the Base Unit Count field) and would therefore use a numbered list here to make it easier to read with more steps.
• I'd clarify that the price per unit is displayed in addition to the sales prices (i.e., it doesn't replace it), just in case (I know you mention it in the example below but still)
• See my general comment about relative paths that go back to "/applications"

Suggested change

	To display a :doc:`price per unit
	<../../../inventory_and_mrp/inventory/product_management/configure/uom>` on the product page, go to
	:menuselection:`Website --> Configuration --> Settings`, scroll to the :guilabel:`Shop - Products`
	section, and enable :guilabel:`Product Reference Price`. Navigate to the product form, set an amount
	in the :guilabel:`Base Unit Count` and :guilabel:`Sales Price` fields.
	To display a :doc:`price per unit </applications/inventory_and_mrp/inventory/product_management/configure/uom>` alongside the sales price on the product page, follow these steps:
	#. Go to :menuselection:`Website --> Configuration --> Settings`, scroll to the :guilabel:`Shop - Products`
	section, and enable :guilabel:`Product Reference Price`.
	#. :ref:`Navigate to the product form <ecommerce/products/create-products>`.
	#. Set the product's :guilabel:`Sales price` and select the unit (e.g., :guilabel:`Units`).
	#. In the :guilabel:`Base Unit Count` field, enter the number of units in the product, then select the relevant unit from the dropdown list.

(or something similar for the last step)

[auva-odoo]

Suggested change

	Keep in mind that the price per unit may be mandatory in some countries.
	The price per unit may be mandatory in some countries.

[auva-odoo]

While this is technically correct, I feel like it makes more say to say this the other way around:

Suggested change

	commonly used. This involves showing the original price crossed out, with the new discounted price
	displayed alongside it.
	commonly used. This involves showing the new discounted price, with the original price crossed out
	alongside it.

[auva-odoo]

Taxes

I think people might want to use this feature for reasons other than B2B vs B2C so I would have a separate h2 section for this, similar to what was in the old version of this doc, and I think I would move it back up to the top of the page

[auva-odoo]

This should go in the B2B access management section and this can be done from the eCommerce app (peope might not have the Contacts app):

Suggested change

	To make sure that only customers with an account and granted access can see the prices, go to the
	Contacts app, click the top search bar, and select the :guilabel:`Archived` filter. Click
	the *public user*, go to the :guilabel:`Sales & Purchase` tab, and add the zero-priced
	pricelist that has a country groups assigned that contains all countries. The public user
	**always** has to stay :guilabel:`Archived`.
	To make sure that only customers with an account and granted access can see the prices, go to :menuselection:`Website --> eCommerce --> Customers`, click the :icon:`fa-caret-down` :guilabel:`(dropdown)`
	icon from the search bar, and select the :guilabel:`Archived` filter. Then, click the :guilabel:`Public user` card, go to the :guilabel:`Sales & Purchase` tab, and add the :ref:`zero-priced pricelist <add a ref>` configured to :ref:`cover all countries <ecommerce/prices/country-groups>`. Keep the :guilabel:`Public User` contact archived at all times.

[auva-odoo]

Suggested change

	(B2B vs. B2C). To select the type of price displayed, go to :menuselection:`Website -->
	(:doc:`B2B <add link when doc is created>` vs B2C). To select the type of price displayed, go to :menuselection:`Website -->

[auva-odoo]

Suggested change

	This setting is website specific, and therefore can be altered for each website within a
	database.
	This setting is website-specific and can therefore be configured independently for every website in the database.

[auva-odoo]

this list item can be moved to the new B2B doc page
and I think we should add that the toggle is available in the website editor, just for good measure