Skip to content

TI-4036: Rename external choice identifiers to external_key - #26

Merged
Joel-Luca merged 1 commit into
masterfrom
fix/ti-4036-external-key
Sep 24, 2026
Merged

Joel-Luca merged 1 commit into
masterfrom
fix/ti-4036-external-key

Conversation

@Joel-Luca

Copy link
Copy Markdown
Member

Rename the untranslated custom-value external_label field to external_key so its purpose as a stable external identifier is explicit. ChoiceMatcher(choices) now matches exact external keys by default; generic label/value matching remains available explicitly.

Update the example app, admin, translations and upgrade documentation. A reversible example migration preserves existing keys/values and replaces the field-scoped uniqueness constraint. Every concrete consumer needs its own corresponding migration before upgraded workers start. This supports RIS PRs #4133 and #4140; no package release is published here.

Validation: 220 custom-field tests passed; the rename migration round trip passed; the final matching-contract rerun passed 119 tests. makemigrations --check --dry-run, Black, isort and gettext compilation passed. Full CI was not run.

Tracking: TI-4036.

@Joel-Luca
Joel-Luca force-pushed the fix/ti-4036-external-key branch from 12a2d3b to 9c4b7a0 Compare September 23, 2026 13:46
@Joel-Luca
Joel-Luca requested a review from a team September 23, 2026 13:49

@all4manu all4manu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, with some minor nits.

Comment thread app/custom_field/tests/test_external_key_migration.py Outdated
Comment thread django_features/locale/de/LC_MESSAGES/django.po Outdated
Comment thread django_features/custom_fields/matching.py
@Joel-Luca
Joel-Luca force-pushed the fix/ti-4036-external-key branch from 9c4b7a0 to 1a00597 Compare September 24, 2026 09:16
@Joel-Luca
Joel-Luca force-pushed the fix/ti-4036-external-key branch from 1a00597 to dd13dac Compare September 24, 2026 09:47
@Joel-Luca
Joel-Luca merged commit 7c1f8a2 into master Sep 24, 2026
1 check passed
@Joel-Luca
Joel-Luca deleted the fix/ti-4036-external-key branch September 24, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants