Skip to content

fix: change requestSystemData to use monitoring.solaredge.com#13

Open
RichieB2B wants to merge 1 commit into
ProudElm:mainfrom
RichieB2B:no-public
Open

fix: change requestSystemData to use monitoring.solaredge.com#13
RichieB2B wants to merge 1 commit into
ProudElm:mainfrom
RichieB2B:no-public

Conversation

@RichieB2B

Copy link
Copy Markdown

requestSystemData stopped working for me but I was able to fix it.

@gjdv

gjdv commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Great! Thanks for this fix. I can confirm that this endpoint works. I do need to add the &locale=en_US to the url to make the rest of the code work (given that my SolarEdge locale is set to nl_NL), though.

@Mariusthvdb

Mariusthvdb commented Jan 9, 2026

Copy link
Copy Markdown

Nice!

while youre at it , maybe base your Pr on this fork by bdraco, so it uses the right coordinator code

https://github.com/bdraco/solaredgeoptimizers

would be nice to have a solid component

or, since this repo is abandoned, suggest the Pr on that fork, so bdraco can release it.
He’s the core HA wizard dev, no better hands to leave this in ;)

edit

nevermind, I hadn't realized this is not the CC itself....

@Mariusthvdb

Copy link
Copy Markdown

Great! Thanks for this fix. I can confirm that this endpoint works. I do need to add the &locale=en_US to the url to make the rest of the code work (given that my SolarEdge locale is set to nl_NL), though.

how do we test this?

@gjdv

gjdv commented Jan 10, 2026

Copy link
Copy Markdown
Contributor

I'm running this package with the changed endpoint (https://github.com/gjdv/packaging_solaredgeoptimizers) without problems since yesterday (repeatedly). I don't use HA integration, so I can't comment on how to merge this change in there.

@29SimonB 29SimonB left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should work

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.

4 participants