-
Notifications
You must be signed in to change notification settings - Fork 11
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 to the new API #8
Comments
Yeah, well, great story from the dev at Solax, but the data is still only updated every 5min, so there is no use to make any change. |
@pieterzen thanks for the information. |
The tokenId sounds like a good idea so there is no login but the data that
comes back from that new call isn't as complete.
While testing yesterday I've noticed that my old tokenId (the one you get
after login) is still valid. So in a way we already have a persisting
token.
Therefor I feel that there is no need to use the new getRealtimeInfo IF you
know a way to store the tokenId after the first login. You think that's
possible?
Regarding the decimals, I believe we should return Watt and WattHour (so
divide by 1000). That would remove the need for rounding.
…On Sat, 31 Oct 2020 at 10:52, Thomas Cys ***@***.***> wrote:
@pieterzen <https://github.com/pieterzen> thanks for the information.
The tokenID is still a better approach than the username/password
combination.
I'll try to look into it when i have some spare time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#8 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFBVXCYZI27SUX6LXW725JTSNPM6DANCNFSM4TEZHXQQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If at all possible, we could rebuild this integration to use the new API. There are reports of login failures, but that's probably related to the latest HA 117 build.
The new API is documented (in the Solax Cloud portal) and uses a generated token instead of username/password.
The text was updated successfully, but these errors were encountered: