custom-components / sensor.owlintuition

A set of sensors to integrate the OWL Intuition devices network
MIT License
11 stars 9 forks source link

Fix issues with sensors not updating after HA 2024.1 upgrade #37

Closed shortbloke closed 9 months ago

shortbloke commented 9 months ago

Fixed #36 - State is directly updated via self._attr_native_value Address deprecation warnings with specific device and state class combinations used in SENSOR_TYPES definitions. Use new unit enumerators, replacing deprecated constants, per: Constant Deprecation

orandaadnaro commented 9 months ago

I patched and can confirm all is working well

As can be seen below - it didn't ultimately work - I should have given it longer for the app to settle in :-)

glpatcern commented 9 months ago

Just leaving some unasked review :)

Thanks for the review @joostlek ! And as you now learned about this integration, I dare asking you a suggestion and/or some pointers about a different issue, which is essentially the only reason why I never pushed this to be part of HA core: any idea how to make an async UDP listening loop in the context of HomeAssistant?

joostlek commented 9 months ago

Let's chat on discord :)

glpatcern commented 9 months ago

@shortbloke all good to merge?

shortbloke commented 9 months ago

@glpatcern Yep, good to go AFAIK ;-)

orandaadnaro commented 9 months ago

I’m new to all this, but I see you’ve merged some new files into the owl custom setup. Does that mean I will need to re run the terminal script on the customisation page in order to get the new stuff installed?

shortbloke commented 9 months ago

@orandaadnaro Assuming you installed this custom component via HACS, the update will be offered there. You might need to restart the HACS addon or HA in order to get to check for updates.

image

orandaadnaro commented 9 months ago

Thanks that update worked as you described - never seen that before - quite a surprise!

Will future updates come this way?

shortbloke commented 9 months ago

@orandaadnaro That's how updates to all HACS components should be delivered. So yes.