OCA / edi

GNU Affero General Public License v3.0
119 stars 301 forks source link

Migration to version 16.0 #670

Open sbidoul opened 1 year ago

sbidoul commented 1 year ago

Todo

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

Modules to migrate

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

john-herholz-dt commented 1 year ago

base_edi #711

hailangvn commented 1 year ago
john-herholz-dt commented 1 year ago

@simahawk , @sbidoul as I started migrating the EDI modules: Does it make sense to merge base_edi and edi_oca? Furthermore I saw a different menu icon in edi_oca then in base_edi. The one in edi_oca looks much better, can we use this for base_edi as well, then I will update my PR.

alexis-via commented 1 year ago

My roadmap for the migration of account_invoice_import to v16: 1) switch from single-file wizard to multi-file wizard 2) allow to share account.invoice.import.config across companies (company_id required=False and account_id field become company_dependent=True) ? I you have some remarks/suggestions about these 2 changes, it's the good time to talk about it.

With @legalsylvain we also started to brainstorm about a re-write of the "update" scenario, but it will probably take time and arrive after the migration to v16.

john-herholz-dt commented 1 year ago

@sbidoul base_edi was merged, I am currently doing edi_oca but it hangs on the tests. See PR #729

Cedric-Pigeon commented 1 year ago

@alexis-via What's the status of account_invoice_import migration ? We'll need it in a really short-term. May be we can help you ? Thanks,

john-herholz-dt commented 1 year ago

edi_backend_partner_oca is PR #745 Waiting for dependency on PR #729 (Tests don't finish, I don't know the reason). Can someone help?

simahawk commented 1 year ago

@simahawk , @sbidoul as I started migrating the EDI modules: Does it make sense to merge base_edi and edi_oca? Furthermore I saw a different menu icon in edi_oca then in base_edi. The one in edi_oca looks much better, can we use this for base_edi as well, then I will update my PR.

Sorry for the late reply. About merging module: no. It has been done like that to have a common entry point for EDI conf, no matter if you use the framework or not. Regarding the icon: yeah, makes sense :+1:

gomezgleonardob commented 1 year ago

Hello ,It is considered to migrate this modules to v16?

https://github.com/OCA/edi/tree/14.0/account_invoice_import https://github.com/OCA/edi/tree/14.0/account_invoice_import_facturx https://github.com/OCA/edi/tree/14.0/account_invoice_import_ubl

And Same Variants

https://github.com/OCA/edi/tree/14.0/account_invoice_import_simple_pdf https://github.com/OCA/edi/tree/14.0/account_invoice_import_invoice2data

alexis-via commented 1 year ago

@gomezgleonardob Most of these modules are already migrated to v16, just have a look at the pull requests.

gomezgleonardob commented 1 year ago

Thanks, I look the PR :D

trisdoan commented 7 months ago

working on sale_order_packaging_import (Please add to the list of modules above.)