search

custom-components / weatheralerts

A sensor that gives you weather alerts from alerts.weather.gov.
MIT License
125 stars 15 forks source link

Only one alert showing up #30

Closed ntilley905 closed 4 years ago

ntilley905 commented 4 years ago

The other day my county had multiple alerts at once (I believe we only got to two), a wind advisory and a tornado watch. However, the component only ever picked up the wind advisory (I assume because it was published first). Is this a limitation of the component or something that could potentially be fixed, or am I doing something wrong?

Config:

- platform: weatheralerts
  state: 'GA'
  zone: '033'
jlverhagen commented 4 years ago

Check the pull request I submitted (https://github.com/custom-components/weatheralerts/pull/29). The problem is the weatheralerts component is only using zone ID alert feeds. And the weather.gov website doesn't appear to be keeping those feeds accurately updated. The county ID alert feeds do appear to be accurately updated. The pull request I submitted adds the ability to use county IDs. Read my comment on the pull request page. The comment gives some instructions and details.

You can either manually apply the changes from the pull request or download my updated version of the sensor.py file from my weatheralerts fork here: https://github.com/jlverhagen/weatheralerts/blob/patch-1/custom_components/weatheralerts/sensor.py

Post Edit: Sorry, I linked to the master version of the sensor.py file instead of the patched version. The link to my forked patch is now correct above.

jlverhagen commented 4 years ago

Sorry, in my reply above I linked to the master sensor.py instead of the patched version. The link is updated to the correct patched sensor.py file.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.