Skip to content

Reuse the model description label in the site form - #1689

Closed
dlq wants to merge 1 commit into
aces:devfrom
dlq:reuse-site-description-label
Closed

dlq wants to merge 1 commit into
aces:devfrom
dlq:reuse-site-description-label

Conversation

@dlq

@dlq dlq commented Oct 27, 2026

Copy link
Copy Markdown

Remove the duplicate English/French sites.new.labels.description entries and let f.label :description resolve the existing shared attributes.description translation. Both languages continue to display “Description”.

Checked the original key’s consumer and Rails attribute resolution, then opened the new-site form in English/French in live Chromium. The label and editable description field passed; no site was created and no JavaScript errors were reported. This is a verified small cleanup, not a general unused-key purge.

Related to #1660. Independent branch based on dev. The code patches were tested in a combined local snapshot with #1677 (0067e3e) and our #1678–#1684 changes; patches applied cleanly. git diff --check passed.

@dlq dlq mentioned this pull request Oct 27, 2026
1 of 6 tasks
@prioux

prioux commented Oct 29, 2026

Copy link
Copy Markdown
Member

I am rejecting this one because even though in French and English the word "Description" is the same, it might not be in other future languages.

@prioux prioux closed this Oct 29, 2026
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