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

Data Structure Changed in firmware v3.88 node crashing #11

Open
BGCDE opened this issue Jan 5, 2023 · 5 comments · May be fixed by #12
Open

Data Structure Changed in firmware v3.88 node crashing #11

BGCDE opened this issue Jan 5, 2023 · 5 comments · May be fixed by #12

Comments

@BGCDE
Copy link

BGCDE commented Jan 5, 2023

I've fixed luxtronik2-ws.js preventing the crash and adding all new attibutes.
I can create a pull request. Although i am a github newbie.

@Bouni
Copy link
Owner

Bouni commented Jan 13, 2023

Can you somehow evaluate that this also works with heatpumps < V3.88?
I want to avoild make it working for you but break it for all other users ...

@Bouni Bouni linked a pull request Jan 13, 2023 that will close this issue
@BGCDE
Copy link
Author

BGCDE commented Jan 14, 2023

No, firmware downgrade is not possible. I asked in htdialog firmware thread, let's wait for feedback. On the other side ait did not change any basics, just adding a 2nd level for 2 item groups as you named them. All other groups have the old format with no change.

@Henriettel
Copy link

Any news on this? Updated my WP today to V3.88 and now i receive no values anymore -.-
If you need help or a tester, let me know

@BGCDE
Copy link
Author

BGCDE commented Feb 24, 2023

Any news on this? Updated my WP today to V3.88 and now i receive no values anymore -.- If you need help or a tester, let me know

No feedback until now. You can manually copy the js file from my fork to confirm.

@Henriettel
Copy link

I tested the file from the PR. Works fine for 5 days now.

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

Successfully merging a pull request may close this issue.

3 participants