community-data-files icon indicating copy to clipboard operation
community-data-files copied to clipboard

Migration to version 18.0

Open OCA-git-bot opened this issue 1 year ago • 8 comments

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-18.0

Modules to migrate

  • [ ] account_payment_unece
  • [ ] account_tax_unece
  • [x] base_bank_from_iban - By @mpascuall - #228
  • [ ] base_currency_iso_4217
  • [x] base_iso3166 - By @ClemRiviere - #220
  • [x] base_unece - By @LucasTran380381 - #216
  • [x] company_sanitary_registry - By @eduezerouali-tecnativa - #242
  • [x] l10n_eu_nace - By @BertVGroenendael - #229
  • [x] l10n_eu_product_adr - By @chaule97 - #208
  • [x] l10n_eu_product_adr_dangerous_goods - By @xaviedoanhduy - #211
  • [x] product_fao_fishing - By @CarlosRoca13 - #230
  • [ ] uom_unece

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

OCA-git-bot avatar Oct 06 '24 14:10 OCA-git-bot

l10n_eu_product_adr_dangerous_goods: https://github.com/OCA/community-data-files/pull/211

xaviedoanhduy avatar Oct 15 '24 07:10 xaviedoanhduy

account_tax_unece https://github.com/OCA/community-data-files/pull/215

LucasTran380381 avatar Nov 13 '24 10:11 LucasTran380381

base_bank_from_iban #217

achulii avatar Nov 19 '24 17:11 achulii

l10n_eu_nace #218

achulii avatar Nov 25 '24 15:11 achulii

I was wondering if it's a good idea to continue to maintain uom_unece in Odoo because, since v17, the account module has a method _get_unece_code() on uom.uom, cf https://github.com/odoo/odoo/blob/18.0/addons/account/models/uom_uom.py#L17 (this method was part of account_edi from v14 to v16).

This method works to get the UNECE code from an uom. First drawback: when you manually add a unit of measure in Odoo, you have to inherit the method _get_unece_code() to set the corresponding UNECE code. But, for me, the major drawback of adopting the code in the account module is that, when you import an order/invoice, you need to find the uom from the UNECE code, and I don't see how you can do that with the code of the account module (without writing horrible code that loops to every uom to get the corresponding UNECE code and then match... too dirty).

So my conclusion is that we should continue to use uom_unece. Do you have the same opinion ?

alexis-via avatar Dec 18 '24 16:12 alexis-via

Yes, I agree.

pedrobaeza avatar Dec 19 '24 12:12 pedrobaeza

base_iso3166 #220

ClemRiviere avatar Dec 23 '24 10:12 ClemRiviere

l10n_eu_nace #229

BertVGroenendael avatar Feb 17 '25 10:02 BertVGroenendael