change BasePowerSensor device_class to ENERGY

SonnenladenGmbH / APsystems-EZ1-API-HomeAssistant

The APsystems EZ1 Integration offers a streamlined interface for interacting with the local API of APsystems EZ1 Microinverters using Home Assistant.

MIT License

77 stars 11 forks source link

change BasePowerSensor device_class to ENERGY #45

Closed JM-Lemmi closed 1 month ago

JM-Lemmi commented 1 month ago

This PR resolves #44.

If there is a reason to keep the device class at power I'd like to hear about it and work out a solution that fits all the needs.

Cheers, JM-Lemmi

mawoka-myblock commented 1 month ago

Thanks for your work, but that's unfortunately not how it works. https://developers.home-assistant.io/docs/core/entity/sensor#available-device-classes

JM-Lemmi commented 1 month ago

After experimenting a bit more the data in the energy dashboard is appearing correctly with the lifetime production. So it must have been an error on my end. Apologies for the useless issue and PR.

mawoka-myblock commented 1 month ago

No problem, happens to the best of us as well ;)