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
Bump gios
library to version 3.1.0
#89044
Conversation
@@ -50,7 +51,7 @@ async def test_sensor(hass: HomeAssistant) -> None: | |||
== CONCENTRATION_MICROGRAMS_PER_CUBIC_METER | |||
) | |||
assert state.attributes.get(ATTR_ICON) == "mdi:molecule" | |||
assert state.attributes.get(ATTR_INDEX) == "bardzo dobry" | |||
assert state.attributes.get(ATTR_INDEX) == "very_good" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can also provide translations for attributes in our strings.json file.
https://developers.home-assistant.io/docs/internationalization/core#entity-state-attributes
Although the frontend currently only supports climate right now, it would be a matter of time before the rest is implemented. The backend supports it fully.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I plan to migrate index
attributes to the separate entities in the next step. Do you think the attributes will be better? And I want to remove name
and station
attributes because they present static data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm always all in favor of less attributes 🎉 😬
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @bieniu 👍
../Frenck
Breaking change
AQI sensor state values are now in English, and the user interface will display their translation for the selected language.
Users should update their automations and scripts.
Proposed change
What's changed:
In the next step, I plan to migrate
index
sensor state attributes to the separate entities and removename
andstation
attributes because they present static data.Changelog: bieniu/gios@2.3.0...3.1.0
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: