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

[17.0][MIG] upgrade_analysis #2826

Merged
merged 81 commits into from
May 23, 2024

Conversation

StefanRijnhart
Copy link
Member

@StefanRijnhart StefanRijnhart commented Feb 2, 2024

Includes 2/3 of #2739 and a small change that makes it easier to use an uninitialized upgrade path.

Used to generate the analysis at OCA/OpenUpgrade#4294

@StefanRijnhart StefanRijnhart added this to the 17.0 milestone Feb 2, 2024
@StefanRijnhart StefanRijnhart force-pushed the 17.0-mig-upgrade_analysis branch from 845f137 to f13629e Compare February 2, 2024 19:43
@StefanRijnhart StefanRijnhart marked this pull request as draft February 2, 2024 19:46
@pedrobaeza pedrobaeza changed the title [17.0][MIG] upgrade analysis [17.0][MIG] upgrade_analysis Feb 2, 2024
@pedrobaeza
Copy link
Member

/ocabot migration upgrade_analysis

@OCA-git-bot OCA-git-bot mentioned this pull request Feb 2, 2024
35 tasks
@StefanRijnhart StefanRijnhart force-pushed the 17.0-mig-upgrade_analysis branch 2 times, most recently from 5eac3a5 to 7f4b8f2 Compare February 2, 2024 19:51
@StefanRijnhart StefanRijnhart marked this pull request as ready for review February 2, 2024 19:54
legalsylvain and others added 21 commits May 23, 2024 10:03
… 13.0) to OCA/server-tools

Based on commit 746b7acbd90d62f9ffe6ee17472a1a3533e36597 (Fri Nov 6 17:18:47 2020 +0100)

Co-authored-by: Stefan Rijnhart <[email protected]>
… make the module installable, add readme folders
…y to select some modules, and improve filters, regarding test module with startswith and endswith pattern
… (replaced by logger), apply pre-commit rules
[IMP] guess upgrade_path, if config is not set, and openupgrade_scripts is available
Given that the records need to be managed in the modules that own them rather in the modules that override them, and that we also have the noupdate data xml for every module, then we left out the lines that where indicating the override of records.
OCA-git-bot and others added 18 commits May 23, 2024 10:03
…ted fields

Odoo 14 introduced the widescale usage of computed fields with readonly=False.
In that case, the compute method functions as a default that can also be used
to compute a value some time *after* the initial creation of the record.

In the OpenUpgrade analysis files, these fields would be misrepresented as
computed fields rather than fields with a default function. This change fixes
that.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-upgrade_analysis
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-upgrade_analysis/
Check "module" is in context when patching init_models
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: server-tools-16.0/server-tools-16.0-upgrade_analysis
Translate-URL: https://translation.odoo-community.org/projects/server-tools-16-0/server-tools-16-0-upgrade_analysis/
The colspan property of some UI elements were not correctly adjusted to
the v16 sytem.

This commits fixes it.
Odoo has deprecated such modules with a pre-hook raising an error, so
they are not installable anymore.

This way, we avoid them to be included in the Install Modules Wizard.
This allows to circumvent Odoo's complaint of

   'openupgrade_scripts/scripts/' is not a valid upgrade directory

when there are no analysis files present yet.
…ed RST file

to allow for very long module name
@StefanRijnhart StefanRijnhart force-pushed the 17.0-mig-upgrade_analysis branch from 6dec835 to 9424923 Compare May 23, 2024 08:07
@StefanRijnhart
Copy link
Member Author

Rebased to fix conflicts in requirements.txt

@pedrobaeza pedrobaeza requested a review from legalsylvain May 23, 2024 09:01
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

diff review (from 16.0)

LGTM. Thanks !

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-2826-by-legalsylvain-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c9774fa into OCA:17.0 May 23, 2024
7 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e46ebf9. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.