homeassistant_ecowitt icon indicating copy to clipboard operation
homeassistant_ecowitt copied to clipboard

Update __init__.py with conversion of deprecated async_get_registry to async_get

Open nguilbault opened this issue 2 years ago • 9 comments

async_get_registry is deprecated and was removed in 2023.5.0. Converted use of the deprecated function to the replacement async_get

nguilbault avatar May 04 '23 16:05 nguilbault

You also need to remove the await on this line

https://github.com/nguilbault/homeassistant_ecowitt/blob/580daadfa84e4206e2f1207d028e504c5a143b0f/custom_components/ecowitt/init.py#L421

bdraco avatar May 05 '23 01:05 bdraco

good catch - that's what I get for assuming that a function named async_get is actually async.

nguilbault avatar May 05 '23 15:05 nguilbault

Thank you! Fixed my problem after upgrading to 2023.5. Although it looks like this project is dead at this point and there is an official Ecowitt integration now, so I guess I'll be migrating soon.

mattdevo1 avatar May 08 '23 04:05 mattdevo1

@nguilbault Help a lot, thx. @mattdevo1 official Ecowitt integration is cloud based :-(

rpajik avatar May 09 '23 14:05 rpajik

@nguilbault Help a lot, thx. @mattdevo1 official Ecowitt integration is cloud based :-(

The official integration is Local Push (not cloud based) - https://www.home-assistant.io/integrations/ecowitt/

Your Ecowitt hub pushes data to Home Assistant locally. There is no cloud involved.

mattdevo1 avatar May 09 '23 17:05 mattdevo1

@nguilbault Help a lot, thx. @mattdevo1 official Ecowitt integration is cloud based :-(

The official integration is Local Push (not cloud based) - https://www.home-assistant.io/integrations/ecowitt/

Your Ecowitt hub pushes data to Home Assistant locally. There is no cloud involved.

Sorry, my fault. thx

rpajik avatar May 09 '23 18:05 rpajik

This repo is dead. Has anyone a good replacement or working fork ?

nliaudat avatar Jun 14 '23 17:06 nliaudat

This repo is dead. Has anyone a good replacement or working fork ?

Have you tried the official integration?

https://www.home-assistant.io/integrations/ecowitt/

I haven't migrated yet. Would be interested to hear if it works for you.

mattdevo1 avatar Jun 15 '23 02:06 mattdevo1

good catch - that's what I get for assuming that a function named async_get is actually async.

Thanks guys - great fix!

DMcGav avatar Jun 29 '23 12:06 DMcGav