garbled1 / homeassistant_ecowitt

Ecowitt Weather Station integration for homeassistant
Apache License 2.0
142 stars 70 forks source link

Update __init__.py with conversion of deprecated async_get_registry to async_get #153

Open nguilbault opened 1 year ago

nguilbault commented 1 year ago

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

bdraco commented 1 year ago

You also need to remove the await on this line

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

nguilbault commented 1 year ago

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

mattdevo1 commented 1 year ago

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.

rpajik commented 1 year ago

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

mattdevo1 commented 1 year ago

@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.

rpajik commented 1 year ago

@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

nliaudat commented 1 year ago

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

mattdevo1 commented 1 year ago

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.

DMcGav commented 1 year ago

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

Thanks guys - great fix!