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

[FR] weather state is now exposed raw and untranslated #2910

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bors-ltd
Copy link
Contributor

As it goes for every exposed entity of any domain from now on.

This concludes uncertainty about the content of "state.state", and will have the nice side effect of putting the development environment on par with the production behaviour.

Fixes #2899

As it goes for every exposed entity of any domain from now on.

This concludes uncertainty about the content of "state.state", and will
have the nice side effect of putting the development environment on par
with the production behaviour.

Fixes home-assistant#2899
'windy': 'Venteux',
'windy-variant': 'Venteux et nuageux',
} %}
{{ weather_condition.get(state.state | string, "") }}
Copy link
Contributor Author

@bors-ltd bors-ltd Jan 24, 2025

Choose a reason for hiding this comment

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

I assumed the future behaviour already, so I don't "lower" like most weather intents do today.

Edit: But I kept their behaviour of having no fallback when the value is unknown/unavailable.

@bors-ltd
Copy link
Contributor Author

FYI, I've got two experimental features in a core branch to help reduce the clutter of that template:

  • Let define includes and macros to share among all possible responses of a given intent. Works pretty well, except it doesn't reload when the source template is reloaded.
  • Moving the Beaufort scale computation to the base Weather integration, so any and all weather entities expose a new "beaufort_scale" attribute as long as wind speed is available. Just needs tests before being exposed to the public.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FR] Weather State not translated
1 participant