search

astrandb / weatherlink

Weatherlink for Home Assistant
MIT License
11 stars 1 forks source link

Break other Weatherlink integration https://github.com/siku2/hass-weatherlink #90

Closed raf10gh closed 10 months ago

raf10gh commented 10 months ago

The problem

If you have the Weatherlink integration from https://github.com/siku2/hass-weatherlink installed and then you install https://github.com/astrandb/weatherlink integration it breaks the one from siku2.

Version of Weatherlink integration having the issue?

0.9.6

Version of Home Assistant Core having the issue?

2023.12.3

Anything in the logs that might be useful for us?

Thanks for your work on this integration. I tested your Weatherlink integration in my test HA environment and liked that it shows stats like Rain this month and Rain this year. My test environment didn't have https://github.com/siku2/hass-weatherlink integration installed. When I installed it into my Production HA environment it cause the other integration not to load. It would be nice if both could work together. Siku2's integration collects the data in realtime using the IP of the Weatherlink station.

Diagnostic data file

No response

Additional information

No response

raf10gh commented 10 months ago

Thanks for your work on this integration. I tested your Weatherlink integration in my test HA environment and liked that it shows stats like Rain this month and Rain this year. My test environment didn't have https://github.com/siku2/hass-weatherlink integration installed. When I installed it into my Production HA environment it cause the other integration not to load. It would be nice if both could work together. Siku2's integration collects the data in realtime using the IP of the Weatherlink station.

astrandb commented 10 months ago

@raf10gh These two integrations hav differnt approaches to collecting data. There are pros and cons with both models. This integration uses the Weatherlink cloud API and therefore supports the entire product range from Davis Weatherlink. The two integrations cannot coexist in the same HA instance. I will make a note of that on the Readme page.