Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to 6.0 issues and questions #101

Closed
DaveCBr opened this issue Jan 5, 2025 · 5 comments
Closed

Update to 6.0 issues and questions #101

DaveCBr opened this issue Jan 5, 2025 · 5 comments

Comments

@DaveCBr
Copy link

DaveCBr commented Jan 5, 2025

After today's update, I see the following issues;

The information page for the driver still indicates it is version 5.0;

I did delete the sensor that is no longer provided and changed the name of the _alerts_alerts sensor, however I am only seeing two sensor variables being updated;

In the instructions, the link to look up ID by state and county does not point to the correct NWS page. I think that NWS changed the page and and it is difficult to get to the state/county listings.

What is the recommended lookup timing? Maybe 5 minutes or even longer? 1 minute is too short.

@DaveCBr DaveCBr changed the title Update to 6.0 issues Update to 5.1 Jan 5, 2025
@DaveCBr DaveCBr changed the title Update to 5.1 Update to 5.1 issues and questions Jan 5, 2025
@DaveCBr DaveCBr changed the title Update to 5.1 issues and questions Update to 6.0 issues and questions Jan 5, 2025
@finity69x2
Copy link
Owner

In the instructions, the link to look up ID by state and county does not point to the correct NWS page. I think that NWS changed the page and and it is difficult to get to the state/county listings.

I just checked the procedure in the readme to find the county/zone code and it all seems to be working as expected for me.

What is the recommended lookup timing? Maybe 5 minutes or even longer? 1 minute is too short.

Are you referring to the update interval configuration?

by default it is 1 minute but you can put whatever you want in the field. If you want it to update every 5 minutes then put in 5 for the time. I like having the fastest update interval since I'm relying on it for severe weather alert notifications. seconds count.

@DaveCBr
Copy link
Author

DaveCBr commented Jan 6, 2025

The issue with the lookup is when there are already a huge amount of alerts. The web page defaults to the alerts and does not show you the ID of where they are from.

Also, I am having an issue seeing the actual alert. I have the old version running as well and it's still working very well. I am probably deleting the new version for now. No reason that I can see to update.

@finity69x2
Copy link
Owner

The issue with the lookup is when there are already a huge amount of alerts. The web page defaults to the alerts and does not show you the ID of where they are from.

I'm not really sure I understand what you are trying to do with that webpage.

All that link is used for in respect to this integration is to be able to find your county or zone id to put into the integration config.

aside from that usage what are you looking there for?

Also, I am having an issue seeing the actual alert

where are you looking for the alert? the integration entity or the NWS webpage?

What version did you update from? There was a big breaking change going from versions prior to v5.0 to v5.0 or later.

@DaveCBr
Copy link
Author

DaveCBr commented Jan 6, 2025

@finity69x2 SO after further digging into the integration I am using, it appears to be the original Weather Alerts custom module from 4 years ago. The version is "0.1.5". ( custom-components/weatheralerts#83 ). The build was deemed to be abandoned about 6 months ago, however is mostly still working. You may have forked that code or been involved with a different ID. There are legacy errors being thrown, but still pushes to our phones, announces through the house, pop ups work and Notifications work for up to 5 alerts. Anyway, I will move over to your code and will start the process of fixing all the above to match what your doing.

@finity69x2
Copy link
Owner

That is a completely different integration than this one. I didn't fork it for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants