search

MickMake / GoSungrow

GoLang implementation to access the iSolarCloud API updated by SunGrow inverters.
https://mickmake.com/
GNU General Public License v2.0
148 stars 42 forks source link

update iSolarCloud\AppService\login\data.go #75

Closed sungrow-firmware closed 9 months ago

sungrow-firmware commented 10 months ago

Change org_id type from string to valueTypes.Integer

Bartleby1980 commented 10 months ago

Hello and sorry for my bad english.

I have the same problem but i cant find the file data.go for change in homeassistant.

I found the file data.go and change the entry. But the change is only on my PC. How can i change the file in homeassistant?

copadewe commented 10 months ago

Wo bitte hast du denn die Datei gefunden? Where did you found the file? I have no idea where to look for.

Bartleby1980 commented 10 months ago

Hallo schön auf deutsch zu antworten.

Die Datei ist unter der folgenden URL zu finden.

https://github.com/sungrow-firmware/GoSungrow/tree/master/iSolarCloud/AppService/login

Ich hab auch das gesamte Projekt heruntergeladen und nach Github-Desktop importiert.

Dann kann ich zwar ein neues Projekt mit der geänderten Datei erstellen, bekomme das in HomeAssistant aber nicht ersetzt.

Von: copadewe @.> Gesendet: Mittwoch, 30. August 2023 14:57 An: MickMake/GoSungrow @.> Cc: Bartleby1980 @.>; Comment @.> Betreff: Re: [MickMake/GoSungrow] update iSolarCloud\AppService\login\data.go (PR #75)

Wo bitte hast du denn die Datei gefunden? Where did you found the file? I have no idea where to look for.

— Reply to this email directly, view it on GitHub https://github.com/MickMake/GoSungrow/pull/75#issuecomment-1699119523 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A53KJ6LGKSRYFTB3QVTZDS3XX42DNANCNFSM6AAAAAA4ECYQGU . You are receiving this because you commented. https://github.com/notifications/beacon/A53KJ6PYLXAJ4UTOALTICLLXX42DNA5CNFSM6AAAAAA4ECYQGWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTFI2A2G.gif Message ID: @.***>

Paraphraser commented 10 months ago

This fix works for me too.

copetogo commented 10 months ago

This fix works for me too.

Hey how do you make change in the json file in home assistant ?

Paraphraser commented 10 months ago

Sorry. I should have been more clear.

I applied your Pull Request, recompiled GoSungrow, and now I can login again. That's all I meant.

I don't use HomeAssistant so I have no idea what to do.

jrbconsulting commented 10 months ago

If the main branch of GoSungrow is fixed, you may need to reinstall or rebuild the add-on in Home Assistant.

copetogo commented 10 months ago

If the main branch of GoSungrow is fixed, you may need to reinstall or rebuild the add-on in Home Assistant.

Yeah I tried rebuild so I guess the code change not yet incorporated . I was thinking if I can edit the json manually via HA terminal

brockgoodall commented 10 months ago

Is anyone able to provide guidance as to how to make this change within Home assistant?

Paraphraser commented 10 months ago

For anyone looking for a way of making this change in Home Assistant, please see this gist.

I'd appreciate feedback.

oleg-d commented 10 months ago

Fixed in 3.0.7