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 #3681

Open
legalsylvain opened this issue Jan 10, 2023 · 19 comments
Open

Migration to version 16.0 #3681

legalsylvain opened this issue Jan 10, 2023 · 19 comments
Milestone

Comments

@legalsylvain
Copy link
Contributor

legalsylvain commented Jan 10, 2023

Pending / done work

@marielejeune
Copy link
Contributor

Hi,
Is there any reason why some modules are not present in openupgrade_scripts/scripts? For example contacts, base_setup, http_routing and web_unsplash?

@legalsylvain
Copy link
Contributor Author

Hi @marielejeune. The scripts files (diff.txt + noupdate.xml) are generated by this module https://github.com/OCA/server-tools/tree/16.0/upgrade_analysis.

I don't know why the modules are missing. You can run this module if you want to investigate.
(note : if it's the first time, you have to run two instances 15 + 16 with upgrade_analysis installed on both, and then make the V16 communicate with the 15. It will create / update all the analysis).

kind regards.

@remytms
Copy link
Contributor

remytms commented May 11, 2023

@legalsylvain I've run again the upgrade_analysis module, with a database with base_setup installed. It creates the analysis files for base, and other module, but not for base_setup. Currently I don't know why.

Is the generation of the analysis file a requirement for writing migration scripts for this module in the OCA process ?

@legalsylvain
Copy link
Contributor Author

legalsylvain commented May 11, 2023

Hi.

  1. I think it's more a topic related to https://github.com/OCA/server-tools/tree/16.0/upgrade_analysis
  2. Yes I saw that sometimes, there is no analysis file, and sometimes, there is an analysis file with minimal text. (nothing to do in the three part). But I don't know why.

Is the generation of the analysis file a requirement for writing migration scripts for this module in the OCA process ?

No. But if there is no file, I think that there is nothing to do.
But you can write a pre- / post- migration file if you see that some scripts are required.

@legalsylvain
Copy link
Contributor Author

Hi all.
FYI, changes on account.tag will be done on 16.0 "stable" version. See : OCA/account-analytic#551 (comment)

I guess that once done, we should regenerate analysis.txt file.

CC : @OCA/openupgrade-maintainers

@pedrobaeza
Copy link
Member

Let's see first how it's handled. Not sure the path they are going to take. I suppose they are going to create an extra module for restoring the missing models and defaults, but it's just a theory.

@duong77476-viindoo
Copy link
Sponsor Contributor

Hi all. FYI, changes on account.tag will be done on 16.0 "stable" version. See : OCA/account-analytic#551 (comment)

I guess that once done, we should regenerate analysis.txt file.

CC : @OCA/openupgrade-maintainers

Are you going to regenerate the analyst fiels soon, i see some of them are outdated (eg: analytic)

@legalsylvain
Copy link
Contributor Author

Hi @duong77476.

Thanks for your all your recent contribution.

Are you going to regenerate the analyst fiels soon, i see some of them are outdated (eg: analytic)

Could you try to do a PR on this subject?
Basically, it's about playing with 2 instances of odoo. (15 and 16) with the upgrade_analysis module installed.

See : https://github.com/OCA/server-tools/tree/16.0/upgrade_analysis#upgrade-analysis

Let me know if you have questions !

@duong77476-viindoo
Copy link
Sponsor Contributor

@legalsylvain Sure no problem, i have run it before back in Odoo 15 Migration

@mostafabarmshory
Copy link

How could I check if account migration is done? can any one help?

@legalsylvain
Copy link
Contributor Author

How could I check if account migration is done? can any one help?

Hi. account (and analytic) migration PR are referenced in this issue.
You can review it and give feedback.

thanks !

@mholthausen

This comment was marked as off-topic.

@pedrobaeza

This comment was marked as off-topic.

@marielejeune
Copy link
Contributor

Hi, I've opened a new PR to finish (I hope) the migration scripts for stock module: #4291

@robinkeunen
Copy link
Contributor

robinkeunen commented Mar 8, 2024

Hi, I see account_qr_code_sepa in the module list at module150-160.rst but not in the list above nor in openupgrade_scripts directory. Is there something I should do to add it ?

Same question for auth_totp_portal.

EDIT: I see the diffs are empty => marking them as done.

@legalsylvain
Copy link
Contributor Author

not in the list above

this is the list of all the merged / pending PR.

EDIT: I see the diffs are empty => marking them as done.

yes. could you make a PR ?

@parsonli

This comment was marked as off-topic.

@duong77476-viindoo

This comment was marked as off-topic.

@legalsylvain

This comment was marked as resolved.

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

No branches or pull requests

9 participants