Fix grammar and clarity in samsungtv/strings.json#169574
Fix grammar and clarity in samsungtv/strings.json#169574chemelli74 merged 3 commits intohome-assistant:devfrom
Conversation
|
Hey there @chemelli74, @epenet, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
There was a problem hiding this comment.
Pull request overview
Improves user-facing copy in the Samsung TV integration’s config flow strings to be clearer and grammatically correct.
Changes:
- Refined invalid host/PIN error phrasing and punctuation.
- Improved conditional wording in setup instructions (“If you have never connected…”).
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR improves user-facing configuration flow strings for the samsungtv integration to make messages clearer and more consistent with shared Home Assistant translations.
Changes:
- Replaced the integration-specific
invalid_hostmessage with the sharedcommon::config_flow::error::invalid_hosttranslation key. - Refined the
invalid_pinerror text for clearer grammar and punctuation. - Improved wording/grammar in the “never connected before” guidance shown during setup.
|
@chemelli74 thank you for your support |
Breaking change
this PR is not a breaking change
Proposed change
This PR fixes a few grammar and wording issues in user-facing strings in
homeassistant/components/samsungtv/strings.json.Type of change
Additional information
Checklist
ruff format homeassistant tests)To help with the load of incoming pull requests: