Repository navigation
[20.0][MIG] base_tier_validation_correction: Migration to 20.0 - #97
Open
Pani-k-folk wants to merge 25 commits into
Open
Pani-k-folk wants to merge 25 commits into
Pani-k-folk wants to merge 25 commits into
Conversation
Currently translated at 100.0% (108 of 108 strings) Translation: server-ux-18.0/server-ux-18.0-base_tier_validation_correction Translate-URL: https://translation.odoo-community.org/projects/server-ux-18-0/server-ux-18-0-base_tier_validation_correction/it/
…e loaded With the changes introduced in odoo/odoo@de056cc#diff-b37c7fd5520c97a29ddc59495779e7be61a66e15e85603928e27b3affb9dc31f, an error was occurring because the change validation on the model was being performed before tearDownClass was executed. Therefore, it has been modified so that the mock changes are applied in each test and cleaned up after each test. This way, the error will no longer appear and the test will run normally.
Currently translated at 100.0% (105 of 105 strings) Translation: tier-validation-19.0/tier-validation-19.0-base_tier_validation_correction Translate-URL: https://translation.odoo-community.org/projects/tier-validation-19-0/tier-validation-19-0-base_tier_validation_correction/nl/
…me Search Name Search used the target model's name search, which is often not what users expect: on journal entries it matches the entry number, still '/' for drafts, so searching "Draft Invoice" found nothing, silently. Replace it with a Documents filter edited with the domain widget, like the "Apply On" domain of tier definitions: filter on partner, date, state... The widget shows how many documents match while editing. Documents without open reviews are still never included. Opened from a document with "Change Reviewer", the filter targets that document. The post-migration carries existing Name Search values over as a filter on the display name, which is what the name search matched, and drops the old column. The manifest version goes to 19.0.1.1.0 with it: pylint-odoo requires the manifest to be at least the version of its migration scripts.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: tier-validation-19.0/tier-validation-19.0-base_tier_validation_correction Translate-URL: https://translation.odoo-community.org/projects/tier-validation-19-0/tier-validation-19-0-base_tier_validation_correction/
Currently translated at 100.0% (106 of 106 strings) Translation: tier-validation-19.0/tier-validation-19.0-base_tier_validation_correction Translate-URL: https://translation.odoo-community.org/projects/tier-validation-19-0/tier-validation-19-0-base_tier_validation_correction/it/
Currently translated at 100.0% (106 of 106 strings) Translation: tier-validation-19.0/tier-validation-19.0-base_tier_validation_correction Translate-URL: https://translation.odoo-community.org/projects/tier-validation-19-0/tier-validation-19-0-base_tier_validation_correction/nl/
Assisted-by: Claude Opus 5.5
OCA-git-bot
added
mod:base_tier_validation_correction
Pani-k-folk
marked this pull request as ready for review
October 12, 2026 08:29
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
12 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.
Depends on: #94
The switch from Name Search (
search_name) to the Documents domain filter (document_domain) is not new here; it comes from the 19.0 history ([IMP] base_tier_validation_correction: Documents filter instead of Name Search).Changes
20.0.1.0.0.migrations/19.0.1.1.0(as the 20.0 migration guide says). Databases still usingsearch_namemust be upgraded to the latest 19.0 first.ir.model.accessis replaced byir.accessin 20.0. Convertedsecurity/ir.model.access.csvtosecurity/ir.access.csv, same xmlids and permissions (crudfor the Tier Correction group,rfor internal users).iconattribute takes a Material Symbols name (Font Awesome is not loaded in the backend):fa-male→person,fa-list→format_list_bulleted,fa-clock-o→schedule.oe_greyno longer exists, replaced bytext-muted.evalon numeric cron field,RUF012noqa on_inherit).test_migrate_name_search, which loaded the deleted 19.0 migration script.document_domainout of the half-width "Find documents" column into its own full-width group, so the domain editor is no longer squeezed.Assisted-by: Claude Opus 5.5 (migration changes were reviewed and tested by me)