search

xlcnd / meteoalarmeu

A 'custom component' for Home-Assistant for Weather Warnings from meteoalarm.eu
GNU General Public License v3.0
20 stars 3 forks source link

Multiple sensors #8

Closed mcshosho closed 3 years ago

mcshosho commented 3 years ago

With binary_sensor defined in configuration.yaml, it was possible to add multiple sensors (for multiple regions). With this new way of defining configuration this is not possible anymore because the unique ID generated at initial config is "meteolaram" and this is not changed even if I rename the initial sensor. Do you think it will be possible to generate a complex unique id (like meteoalarm+country+region)? In this way multiple sensors can be added.

xlcnd commented 3 years ago

Initially I went for the possibility of multi sensors... However, when I evaluated the implications for future developments, I saw that it will make the code very complicated. So for now I think this is the best solution.

But I will take a look again!

nibblerrick commented 3 years ago

I would like to use multiple sensors, too. So just to let you know there are isn't only one who wants it. For now I am happy to have one working sensor (as for me the official integrations isn't working for a long time, too). Thanks!