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

Add english translations, fix several hassfest errors #179

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

rhammen
Copy link
Contributor

@rhammen rhammen commented Oct 30, 2023

Fixes these hassfest erros:
Error: R] [MANIFEST] Invalid manifest: extra keys not allowed @ data['homeassistant']. Got '2023.1.0'
extra keys not allowed @ data['entity']['sensor']['error_reason']['cause']. Got {'name': 'Cause'}
extra keys not allowed @ data['entity']['sensor']['error_reason']['remedy']. Got {'name': 'Remedy'}

Fix Error: R] [MANIFEST] Invalid manifest: extra keys not allowed @ data['homeassistant']. Got '2023.1.0'
Add english Cause & Remedy translations.

Fixes the following hassfest errors:
extra keys not allowed @ data['entity']['sensor']['error_reason']['cause']. Got {'name': 'Cause'}
extra keys not allowed @ data['entity']['sensor']['error_reason']['remedy']. Got {'name': 'Remedy'}
Import latest changes to resolve merge conflicts
Copy link
Owner

@BenPru BenPru left a comment

Choose a reason for hiding this comment

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

Thanks

@BenPru
Copy link
Owner

BenPru commented Nov 7, 2023

Sorry for the delay. I have created a new branch luxtronik_by_ids.
I change the key management to simplify the enums. No double string and int enum. Just 3 IntEnums for Params, Calcs and Visis.
It is not running and I'm still working on it.

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.

2 participants