Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migration to version 16.0 #670

Open
6 of 18 tasks
sbidoul opened this issue Oct 3, 2022 · 12 comments
Open
6 of 18 tasks

Migration to version 16.0 #670

sbidoul opened this issue Oct 3, 2022 · 12 comments
Labels
help wanted no stale Use this label to prevent the automated stale action from closing this PR/Issue. work in progress
Milestone

Comments

@sbidoul
Copy link
Member

sbidoul commented Oct 3, 2022

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

@sbidoul sbidoul added help wanted no stale Use this label to prevent the automated stale action from closing this PR/Issue. work in progress labels Oct 3, 2022
@sbidoul sbidoul added this to the 16.0 milestone Oct 3, 2022
@john-herholz-dt
Copy link

base_edi #711

@hailangvn
Copy link
Contributor

  • account_invoice_import: WIP
    (Please add to the list of modules above.)

@john-herholz-dt
Copy link

@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
Copy link
Contributor

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
Copy link

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

@Cedric-Pigeon
Copy link

@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
Copy link

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
Copy link
Contributor

@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 👍

@alexis-via
Copy link
Contributor

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

@gomezgleonardob
Copy link

Thanks, I look the PR :D

@simahawk simahawk pinned this issue Aug 11, 2023
@trisdoan
Copy link

trisdoan commented Feb 16, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted no stale Use this label to prevent the automated stale action from closing this PR/Issue. work in progress
Projects
None yet
Development

No branches or pull requests

8 participants