search

jesserizzo / envoy_reader

MIT License
37 stars 26 forks source link

User configurable user/pass and to expose Last Reported Date for Inverters #11

Closed gtdiehl closed 4 years ago

gtdiehl commented 5 years ago

Some Envoy devices have a different username and password for the Inverter page. It would be nice to give the user the option to use the default of 'envoy' for the username and the last six digits of the serial number for the password. Or override the default with their own username and password.

I updated the API to expose the Last Reported date for each Inverter. This is helpful to know when an inverter fails to report, as in the case of a PowerLine communication problem or actual inverter failure.

When running the envoy_reader from the console, I added user inputs for the new user/pass authentication config for the inverter data.

I've update the Home Assistant sensor.py code to use these changes as well. The updated sensor can be found at my HA fork: HA Enphase Sensor

I haven't opened a PR in HA yet. Will wait until this PR is merged and the envoy_reader is rev'd so I can update the manifest.json file as well.

jesserizzo commented 4 years ago

@gtdiehl I never saw this PR until now. I just merged it in. Let me know when you have the PR for Home Assistant ready to go and I'll release a new version of envoy_reader on PyPi

gtdiehl commented 4 years ago

@jesserizzo Thank you for merging the changes! I've made my changes with the latest Home Assistant dev branch. Can you release a new version of envoy_reader on PyPi? As I have to include the new version of envoy_reader in the manifest.json for the HA sensor.

jesserizzo commented 4 years ago

@gtdiehl Ok, just uploaded it. Version 0.10.0. Thanks.