squishykid / solax

🌞 Solax Inverter API Wrapper
MIT License
104 stars 63 forks source link

New Wifi dongle support #90

Closed rupertnash closed 8 months ago

rupertnash commented 2 years ago

Various issues/comments suggest that new pocket wifi dongles have to be accessed via a POST with the request in the data rather than URL.

This makes the detection harder and perhaps requires the cartesian product of inverters and dongles?

Issue #80 prompted me to split out the method of access into a base class, but I would expect that the data received from different registers will need a different mapping onto sensor descriptions. One could hope that this will be the same mapping from indexes to meanings for the same model inverter meaning that part can live as a mixin and then the concrete types can just inherit from both InverterPostData and that?

I'm afraid I don't have the time right now to contribute but if tagged in a PR I am happy to run a test against my inverter (X1 Hybrid G4 with new wifi dongle - it returns gives version 3.003.02)

jaykijay commented 2 years ago

See also #88. The new Method via POST request works for my X1 Mini, too, i patched it myself in my own fork.

squishykid commented 8 months ago

resolved in 96