Skip to content

fix(translations): fix fr.json structure causing all French strings to be ignored#112

Merged
kmich merged 1 commit into
mainfrom
claude/eloquent-curie-14a4a8
Jun 26, 2026
Merged

fix(translations): fix fr.json structure causing all French strings to be ignored#112
kmich merged 1 commit into
mainfrom
claude/eloquent-curie-14a4a8

Conversation

@kmich

@kmich kmich commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • indoor_rooms_opt and all subsequent options flow steps were nested inside features_opt.data_description instead of being siblings of features_opt in options.step
  • Home Assistant silently discards a malformed translation schema and falls back to English, which is exactly what was reported in French translation no longer applied after recent updates #111
  • Also removes a stale extra } at the end of the file left from before PR Update fr.json #109

The JSON was syntactically valid (which is why it was hard to spot), but structurally wrong at the schema level HA validates against.

Test plan

  • Reload the integration with a French HA instance and confirm options flow labels appear in French
  • Confirm python -c "import json; json.load(open('custom_components/ws_core/translations/fr.json', encoding='utf-8'))" exits cleanly

Closes #111

🤖 Generated with Claude Code

…o be ignored

indoor_rooms_opt and all subsequent options flow steps were nested inside
features_opt.data_description instead of being siblings of features_opt
in options.step. Home Assistant silently discarded the malformed structure
and fell back to English. Also removes a stale extra closing brace that
was left over before PR #109.

Closes #111

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@kmich
kmich merged commit 1562ad5 into main Jun 26, 2026
10 checks passed
@kmich
kmich deleted the claude/eloquent-curie-14a4a8 branch July 9, 2026 06:13
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.

French translation no longer applied after recent updates

1 participant