T3067 my compassion 18.0 migration#2108
Conversation
- partner_salutation: add is_shown_on_public_forms on res.partner.title with a
- curation column; seed Mister and Madam via a post-migration (version 18.0.1.1.0).
- sponsorship_compassion: host legal_agreement_date on res.partner (moved out of
the deleted website_sponsorship) and show it on the partner form.
There was a problem hiding this comment.
Code Review
This pull request introduces a new boolean field is_shown_on_public_forms to partner titles along with a post-migration script to seed default values. It also adds a legal_agreement_date field to partners, updates tests to use TransactionCase instead of SavepointCase, and corrects a relativedelta import. Regarding feedback, it is recommended to change the legal_agreement_date field from Datetime to Date to avoid potential timezone-shifting bugs.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Confidence Score: 5/5The touched areas are narrow and the changes align with the migration and test-compatibility goals. No correctness issues were identified in the reviewed changes, and the updates are covered by targeted model, migration, view, and test adjustments.
What T-Rex did
Reviews (1): Last reviewed commit: "[MIG] T3067: fix tests on v18" | Re-trigger Greptile |
[MIG] T3067: shared fields for the MyCompassion portal (Odoo 18)
This PR adds two shared fields that the MyCompassion portal needs after the old
website_sponsorshipmodule was removed. The fields used to live in that module,so they had to move to a base module that stays.
Please merge this PR first. The other two PRs below depend on these fields.
Key changes
is_shown_on_public_formsonres.partner.title.This flag decides which titles (Mister, Madam, ...) are shown in public forms.
A post-migration script seeds Mister and Madam so they keep showing.
legal_agreement_dateontores.partner(it lived in the deleted
website_sponsorship) and show it on the partner form.Related PRs (please review together)
This is one part of a 3-repo migration. The reviewer of this PR is also concerned
by the other two:
Merge order: compassion-modules first, then compassion-website, then compassion-switzerland.