Skip to content

Conversation

@chga-odoo
Copy link
Contributor

@chga-odoo chga-odoo commented Jan 27, 2026

  • Improved Industry data and dependencies (quality)
  • Moved Brown Beer and Lager Beer to Brewed category
  • Added pH control QCP to data
  • Enabled purchase for keg 20L products and set bottler vendor price to 20
  • Added tax category code Z to excise taxes
  • Added tax category code O to deposit taxes

Task ID: 5782423

@robodoo
Copy link
Collaborator

robodoo commented Jan 27, 2026

Pull request status dashboard

@chga-odoo chga-odoo changed the title [IMP] industry, excise, deposit: data improvements & Peppol tax fixes [ADD] .*: data improvements & tax fixes Jan 27, 2026
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you bump the version

@chga-odoo chga-odoo force-pushed the saas-19.1-update-micro-brewery-chga branch 2 times, most recently from d302b8f to 909cd6f Compare January 28, 2026 13:02
@dhrs-odoo
Copy link
Contributor

Hello @vava-odoo
It's ready for your review
Thanks 😇

Copy link
Collaborator

@vava-odoo vava-odoo left a comment

Choose a reason for hiding this comment

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

Thanks, please see my comments below. Please also retarget to 19.0

@@ -11,54 +11,62 @@
<field name="name">DEP 0.1</field>
<field name="type_tax_use">sale</field>
<field name="tax_group_id" ref="deposit_tax_group"/>
<field name="ubl_cii_tax_category_code">O</field>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This field is defined in account_edi_ubl_cii, which is not in the dependency tree of this industry. It probably is installed with l10n, but none is provided by default. Could you explain why it works here? And why it would in SaaS databases?
Other way would be to write only if the module is installed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

account_edi_ubl_cii is in dependency tree of this industry(excise and deposit). that's why its work here and according to task requirement, category code added here in both module excise and deposit.

@@ -15,6 +15,7 @@
<field name="amount">0</field>
<field name="type_tax_use">sale</field>
<field name="tax_group_id" ref="excises_tax_group"/>
<field name="ubl_cii_tax_category_code">Z</field>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder: should the "no excises" tax be visible? Maybe to discuss with Pierre to be sure

Choose a reason for hiding this comment

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

@vava-odoo & @chga-odoo
if we could i'd like it to be removed / hidden
however to my understanding the mapping of taxes based on fiscal position makes a replacement to a tax, not to a void
therefore if there is any other way i'm in, on the SOL and INVL as well as the tax group if no other excise tax apply

@@ -42,4 +42,20 @@
<field name="team_id" ref="quality.quality_alert_team0"/>
<field name="picking_type_ids" model="stock.warehouse" eval="[(4, obj().env.ref('stock.warehouse0').manu_type_id.id)]"/>
</record>
<record id="quality_point_8" model="quality.point">
Copy link
Collaborator

Choose a reason for hiding this comment

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

in demo/

@chga-odoo chga-odoo changed the base branch from saas-19.1 to 19.0 January 29, 2026 08:42
@chga-odoo chga-odoo changed the base branch from 19.0 to saas-19.1 January 29, 2026 08:46
@chga-odoo chga-odoo changed the base branch from saas-19.1 to 19.0 January 29, 2026 08:52
@chga-odoo chga-odoo force-pushed the saas-19.1-update-micro-brewery-chga branch from 909cd6f to 2fb1d4f Compare January 29, 2026 08:52
- Improved Industry data and dependencies (quality)
- Moved Brown Beer and Lager Beer to Brewed category
- Added pH control QCP to data
- Enabled purchase for keg 20L products and set bottler vendor price to 20
- Added tax category code Z to excise taxes
- Added tax category code O to deposit taxes

Task ID: 5782423
@chga-odoo chga-odoo force-pushed the saas-19.1-update-micro-brewery-chga branch from 2fb1d4f to 698826f Compare January 29, 2026 10:35
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