Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error Validation of translation placeholders for localized (pl) string #1515

Closed
3 tasks
DeimicUser opened this issue Dec 11, 2024 · 1 comment · Fixed by #1518
Closed
3 tasks

Error Validation of translation placeholders for localized (pl) string #1515

DeimicUser opened this issue Dec 11, 2024 · 1 comment · Fixed by #1518
Assignees
Labels
new bug incoming bug issue

Comments

@DeimicUser
Copy link

Prerequisites

  • Model name of your Devices
  • Output from Home Assistant Developer Tools state e.g.
  • Output from Home Assistant Device Diagnostic from BT
{
  YOUR DEVICE DIAGNOSTICS JSON OUTPUT HERE
}

Description

Rejestrator: homeassistant.helpers.translation
Źródło: helpers/translation.py:285
Pierwsze zdarzenie: 19:46:11 (2 zdarzenia)
Ostatnio zalogowany: 19:46:11

Validation of translation placeholders for localized (pl) string component.better_thermostat.config.step.advanced.description failed: (set() != {'trv'})
Validation of translation placeholders for localized (pl) string component.better_thermostat.options.step.advanced.description failed: (set() != {'trv'})

Steps to Reproduce

Expected behavior:

Actual behavior:

Versions

2024.12.2
1.7.0-beta1

Additional Information

@DeimicUser DeimicUser added the new bug incoming bug issue label Dec 11, 2024
@folfy
Copy link
Collaborator

folfy commented Dec 17, 2024

The string is supposed to show the TRV name (placeholder for the variable):
"description": "Advanced configuration {trv}\n***Info about calibration types: https://better-thermostat.org/configuration#second-step***",
The polish translation doesn't mention it:
"description": "Zaawansowana konfiguracja\n\n**Informacja o typach kalibracji: https://better-thermostat.org/configuration#second-step** ",

Will make a PR for fixing it

folfy added a commit to folfy/better_thermostat that referenced this issue Dec 17, 2024
@folfy folfy mentioned this issue Dec 17, 2024
6 tasks
folfy added a commit to folfy/better_thermostat that referenced this issue Dec 17, 2024
folfy added a commit to folfy/better_thermostat that referenced this issue Dec 19, 2024
folfy added a commit to folfy/better_thermostat that referenced this issue Dec 19, 2024
KartoffelToby added a commit that referenced this issue Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new bug incoming bug issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants