Skip to content

[ADD] product_catalog_tree_price_taxes_included, product_price_taxes_included, product_ux: show order price with taxes in catalog and hide extra price columns by default#900

Closed
gal-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-t-63584-gal-v3

Conversation

@gal-adhoc
Copy link
Copy Markdown
Contributor

Changes

  • [ADD] product_catalog_tree_price_taxes_included: show order price with taxes in product catalog
  • [IMP] product_price_taxes_included: hide taxed_lst_price column by default in product list
  • [IMP] product_ux: hide pricelist_price column by default in product list

Related

Task: #63584
Branch: 19.0-t-63584-gal-v3

@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

@gal-adhoc gal-adhoc force-pushed the 19.0-t-63584-gal-v3 branch from 2b9efe2 to f34fda2 Compare May 27, 2026 19:51
@gal-adhoc gal-adhoc requested a review from Copilot May 27, 2026 19:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new module product_catalog_tree_price_taxes_included that displays product catalog prices with taxes included (in both list and kanban views), and updates two existing modules to hide their newly added price columns by default via the optional="hide" attribute.

Changes:

  • New module computing product_catalog_price_taxed based on order context (currency, partner, fiscal position, company taxes).
  • New list and kanban view inheritances showing the tax-included price.
  • Set optional="hide" on pricelist_price and taxed_lst_price columns in existing product tree views.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
product_ux/views/product_product_views.xml Hide pricelist_price column by default.
product_price_taxes_included/views/product_product_views.xml Hide taxed_lst_price column by default.
product_catalog_tree_price_taxes_included/init.py Module init importing models.
product_catalog_tree_price_taxes_included/manifest.py Manifest declaring deps and view data; auto-install.
product_catalog_tree_price_taxes_included/models/init.py Imports product_product.
product_catalog_tree_price_taxes_included/models/product_product.py Adds computed product_catalog_price_taxed field.
product_catalog_tree_price_taxes_included/views/product_product_views.xml List and kanban view inheritances to display the taxed price.

Comment thread product_catalog_tree_price_taxes_included/models/product_product.py
Comment thread product_catalog_tree_price_taxes_included/__manifest__.py
Comment thread product_catalog_tree_price_taxes_included/models/product_product.py Outdated
@gal-adhoc gal-adhoc force-pushed the 19.0-t-63584-gal-v3 branch from f34fda2 to 7d48817 Compare May 27, 2026 19:53
…included, product_ux: show order price with taxes in catalog and hide extra price columns by default
@gal-adhoc gal-adhoc force-pushed the 19.0-t-63584-gal-v3 branch from 7d48817 to 0433f71 Compare May 27, 2026 20:02
@matiasperalta1
Copy link
Copy Markdown
Contributor

@roboadhoc r+ bump

@roboadhoc roboadhoc closed this in 25f923e May 28, 2026
roboadhoc added a commit that referenced this pull request May 28, 2026
…catalog_tree_price_taxes_included 19.0.1.1.0, product_ux 19.0.1.1.0

Merged: #900
@roboadhoc roboadhoc deleted the 19.0-t-63584-gal-v3 branch May 28, 2026 14:45
@roboadhoc roboadhoc added the 18.1 label May 28, 2026
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