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.

Sign up for GitHub

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

Jump to bottom

evcc binding - gridPower is now grid.power #18148

Open
willyfromtheblock opened this issue Jan 21, 2025 · 4 comments · May be fixed by #18199
Open

evcc binding - gridPower is now grid.power #18148

willyfromtheblock opened this issue Jan 21, 2025 · 4 comments · May be fixed by #18199
Labels
bug An unexpected problem or unintended behavior of an add-on

Comments

@willyfromtheblock
Copy link

willyfromtheblock commented Jan 21, 2025 •
edited
Loading

evcc updated their API with version 0.133.0 Latest

channels:
  - id: general#gridPower
    channelTypeUID: evcc:gridPower
    configuration: {}

is no longer correct, since it is trying to read gridPower, it should be grid.power now

Image
@willyfromtheblock willyfromtheblock added the bug An unexpected problem or unintended behavior of an add-on label Jan 21, 2025
@MikeTheTux
Copy link
Contributor

MikeTheTux commented Jan 27, 2025 •
edited
Loading

Ack! This is the related evcc PR that is causing the incompatibility:

@willyfromtheblock
Copy link
Author

Okay, I will verify later and close this issue. Thank you.

@MikeTheTux
Copy link
Contributor

Okay, I will verify later and close this issue. Thank you.

Sorry, I was imprecise with my comment. This is the EVCC PR that is causing the incompatibility. Not the needed openHAB binding change. Updated the comment ...

DanielSan85 pushed a commit to DanielSan85/openhab-addons that referenced this issue Jan 29, 2025
Fix missing grid readings after evcc breaking change in 0.133.0
0de0566 
Fixes openhab#18148

Signed-off-by: Daniel Kötting <[email protected]>
@DanielSan85 DanielSan85 linked a pull request Jan 29, 2025 that will close this issue
Open
@DanielSan85
Copy link

I had the same issue after upgrading to 0.133.0.
I have just created a PR with a fix to this bug.

DanielSan85 pushed a commit to DanielSan85/openhab-addons that referenced this issue Jan 29, 2025
Reimplemented gridPower to keep the binding backwards compatible
b842681 
Fixes openhab#18148

Signed-off-by: Daniel Kötting <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[evcc] Fix missing grid readings after evcc breaking change in 0.133.0 DanielSan85/openhab-addons
3 participants
@willyfromtheblock @DanielSan85 @MikeTheTux