Skip to content

Norwegian Translation #295

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

andersop91
Copy link

Added translation for Norwegian Bokmål and Norwegian Nynorsk. Translation done by ChatGPT and revised afterwards.
Added translation key for binary_sensor.charger entity. This to support easy identification of the entity from other integration (https://github.com/dirkgroenen/hass-evse-load-balancer).

Added translation key for charger entity, to support easy identification of current entity from other integration.
Norwegian Bokmål
Norwegian Nynorsk
@steinmn
Copy link
Contributor

steinmn commented Aug 22, 2025

As mentioned on dirkgroenen/hass-evse-load-balancer#52, I think you can do this a lot simpler without relying on the detour through binary_sensor.charger.

We are gradually trying to align with HA-core code-standards, and entities with a large number of attributes are quite unpopular among the core team as far as I can tell. The binary_sensor.charger-entity is a leftover from the pre-0.7 zaptec integration, and may be removed (or at least be stripped of a lot of attributes) in a future update.

@sveinse
Copy link
Collaborator

sveinse commented Aug 22, 2025

As mentioned on dirkgroenen/hass-evse-load-balancer#52, I think you can do this a lot simpler without relying on the detour through binary_sensor.charger.

We are gradually trying to align with HA-core code-standards, and entities with a large number of attributes are quite unpopular among the core team as far as I can tell. The binary_sensor.charger-entity is a leftover from the pre-0.7 zaptec integration, and may be removed (or at least be stripped of a lot of attributes) in a future update.

Oh, it is? Do you have links to this? Because these special binary_sensors we have for each charger and installation are a neat way to collect arbitrary information from zaptec without needing a SW update to access new entities. Let's move this discussion to a separate issue, please.

Comment on lines 77 to 78
name="Charger",
translation_key="charger", # added to facilitate for other integrations
Copy link
Collaborator

Choose a reason for hiding this comment

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

This change puzzles me a bit, please elaborate why. What's wrong with the old? As the original comment stated, the exact point is not to have translation on this entity. Moreso, I cannot see any translation for "charger", so I don't quite get this.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for the comment. I'll sure do elaborate -> Check this comment here. The need of the translation key emerged as the other integration already used translation_keys to identify entities.
I guess the actual translation files (.json's) could have been posted in a separate request, to avoid mixing the usage.

Copy link
Collaborator

Choose a reason for hiding this comment

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

That would be nice to move that to another PR, because I think the translation itself is ready for merge.

@steinmn
Copy link
Contributor

steinmn commented Aug 22, 2025

Oh, it is? Do you have links to this? Because these special binary_sensors we have for each charger and installation are a neat way to collect arbitrary information from zaptec without needing a SW update to access new entities. Let's move this discussion to a separate issue, please.

#297

@andersop91 andersop91 changed the title Norwegian Translation + Charger translation key Norwegian Translation Aug 22, 2025
Copy link
Contributor

@steinmn steinmn left a comment

Choose a reason for hiding this comment

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

Bittelitt småpirk, men eg veit ikkje heilt om eg er einig med meg sjølv heller, så går nok an å overtala meg om du har gode argument.

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.

3 participants