Integration is using deprecated DEVICE_CLASS_* constants

[frenck]

This custom integration uses deprecated DEVICE_CLASS_* constants in its codebase.

The DEVICE_CLASS_* constants have been deprecated and replaced in Home Assistant Core 2021.12 (over a year ago). I would highly suggest updating/migrating this integration to the new enums.

For example, for the device classes supported by the sensor platform, there is now a SensorDeviceClass enum. So if a sensor previously used the DEVICE_CLASS_ENERGY constant, it should now use SensorDeviceClass.ENERGY. Other platforms (like binary_sensor, and number) provide similar enumerations for their supported device classes.

The migration thus only consists of replacing constants with an enumeration member and is, therefore, very low impact and should be fairly straightforward.

If I can help resolve any questions regarding this change or migration, feel free to ask or respond to this issue. I'm happy to help!

Kindest regards,

../Frenck

[Manbearpig92]

Hey Frenck,

I just got the wifi module yesterday. I also just updated home assistant this morning but I keep getting this error message every-time i try to integrate it:

Error
Config flow could not be loaded: {"message":"Invalid handler specified"}

I am fairly new to home assistant so your description above regarding enums is kind of foreign to me lol

[frenck]

@Manbearpig92 My report has probably nothing to do with what you are looking for / asking for.

My issue report is about the integration internals (the actual code), which isn't broken but does use some outdated features (which, again, aren't broken at this point).

If you have an issue, please I suggest you create a new one :)

../Frenck

[Manbearpig92]

Oh yeah you're right. I did manage to get it integrated. Were you able to expose the light and pump speed to HA?

[Manbearpig92]

Oh yeah you're right. I did manage to get it integrated. Were you able to expose the light and pump speed to HA?

[scottyphillips]

Good morning @frenck,

1. Thank you so much for Home Assistant!
2. I think my PR Fix up deprecated constant warnings. #38 addresses your issue.
3. Totally off topic - Would you be interested if I brought across my custom component ECHONETLite as a core component? It enables support for a Japanese Home Automation standard ECHONETlite. I think its a major standard over there.