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

Update default url to openapi.growatt.com #77

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

marcovtwout
Copy link
Contributor

Fixes #74

Based on the similar change applied to Home Assistant: home-assistant/core#109122
The PR author there received these URLs from the Growatt IT department.

I also found this document online, probably originating from Growatt as well, mensioning the same URLs:
https://www.showdoc.com.cn/262556420217021/1494054927748011
image

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@indykoning indykoning merged commit 2bfebf7 into indykoning:master Feb 14, 2024
@indykoning
Copy link
Owner

Awesome! I'm glad we got an official response on which urls to use for this

@simonf2004
Copy link

Does this mean that calls to the updated urls are not going to get blocked? Are these urls specifically for this purpose?
Thanks!

@marcovtwout
Copy link
Contributor Author

@simonf2004 We don't know for sure as there is no officially published information for this. But few bug reports came in since the change so it looks OK for now.

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.

405 Client Error
3 participants