[18.0][MIG] upgrade_analysis: Migration to v18 - #3108
Merged
Conversation
… 13.0) to OCA/server-tools Based on commit 746b7acbd90d62f9ffe6ee17472a1a3533e36597 (Fri Nov 6 17:18:47 2020 +0100) Co-authored-by: Stefan Rijnhart <stefan@opener.am>
… make the module installable, add readme folders
…y to select some modules, and improve filters, regarding test module with startswith and endswith pattern
Co-authored-by: Stefan Rijnhart <stefan@opener.am>
… (replaced by logger), apply pre-commit rules
…using upgrade-path
[IMP] guess upgrade_path, if config is not set, and openupgrade_scripts is available
Co-authored-by: David Beal <david.beal@akretion.com>
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.
Includes minor fixes.
Just in case.
pedrobaeza
approved these changes
Nov 5, 2024
pedrobaeza
left a comment
Member
There was a problem hiding this comment.
Code review + see the result
You may squash a bit administrative commits.
legalsylvain
approved these changes
Nov 5, 2024
legalsylvain
left a comment
Contributor
There was a problem hiding this comment.
quick diff review. LGTM. thanks for porting this module !
…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.
Check "module" is in context when patching init_models
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
as odoo core was changed in a way that optimized away our logging
sergio-teruel
force-pushed
the
18.0-mig-upgrade_analysis
branch
from
November 5, 2024 15:31
cd4b150 to
a172892
Compare
Contributor
Author
|
Squashed administrative commits!! |
Member
|
/ocabot merge nobump |
Contributor
|
What a great day to merge this nice PR. Let's do it! |
Contributor
|
Congratulations, your PR was merged at 197ddef. Thanks a lot for contributing to OCA. ❤️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cc @Tecnativa
ping @legalsylvain @pedrobaeza