forked from OCA/openupgradelib
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Master merge from upstream 20240713 #14
Merged
Merged
Conversation
This file contains 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
changed on 17.0 (odoo/odoo@5bf1207)
[FIX] load_data: update to version 17.0
Avoid record values which html is empty using the same rules that odoo.tools.is_html_empty uses. This way we can simplify the call avoiding a default domain as well. In any case, we can use that domain to refine certain models. TT46020
…_4to5 [IMP] BS4->BS5: performance
The improvement made in 40741d9 couldn't deal with json columns of translatable fields. In any case a more simpler approach can be took if we simply discard all the records which don't even have a `class` attribute in their html code, meaning that we don't need to make the Bootstrap migration at all. TT46020
…_4to5-not_html_empty_where_clause [FIX][IMP] _convert_field_bootstrap_4to5_orm: translatable fields
Avoid convertion of values which aren't set saving a handfull of useless queries.
[IMP] convert_field_to_html: pre-clean empty values
To have a hint on possible bottleneck issues for filters with heavy queries. We can disable it setting the new `verbose` parameter to `False`.
…ters-probing [IMP] disable_invalid_filters: log filter probes
We're making use of core's `_get_translation_upgrade_queries` which aside from provide the proper queries to upgrade the translated fields to the new json structure tries to cleanup the former ir_tanslation records. In OpenUpgrade we try to keep historic info after migration so we're able to quickly react to possible inconsistencies. Besides that, we need to rename the `ir_translation` table if we want to work with this method. TT49615
…_jsonb-delete-records [FIX] migrate_translations_to_jsonb: avoid ir_translation terms cleanup
Odoo's core method doesn't convert columns to jsonb. We need to make sure is done befor migrating the translations. TT49615
…_jsonb-convert-column [FIX] migrate_translations_to_jsonb: convert columns to jsonb
The passed value is a recordset, so we should assign the ID of it. Solves OCA/OpenUpgrade#4419
…ds-merge_m2o_orm [FIX] openupgrade_merge_records: Adjust m2o value with ORM
master_merge_from_upstream_20240713
28a87dc
to
5b2a477
Compare
5b2a477
to
3b3a817
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.