-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
FEAT: Notifications In Frigate Frontend #3243
Conversation
1401007
to
65e8691
Compare
Here are some screenshots showing the current UI: Note that the icon will only show when there ARE notifications returned by the API, otherwise it will not show at all. The yellow color (which is set to always be the color) is intended to convey that there are warnings but if this isn't a good choice then happy to change this. |
This is fantastic! Wondering if such information can be sent over via MQTT. HASS can listen to this and send a apple or android notification!. Thoughts? |
Thanks! They're not really fired off live right, it's just a calculated list. I'm not sure when HASS would receive them unless it was just a problem sensor that always had a value that HASS fetched |
Considering I just filled up a disk where my recordings live on my server and basically crashed it for a few hours.... id love notifications via the gui! I rarely check in on my server ( sigh.... ) but I check frigate daily for new events. |
Good idea :) |
I think this is ready for initial review / comments Also to be clear, I have no preference on if it goes into |
Having the update check / info is great. The link could probably be somewhat dynamic; either go to the releases page or wherever depending on what level of release you are at or stage of development Frigate is at |
Right, that one is easy, but the dead camera, low storage, etc. notifications have no link that would work for all install types |
I see the dead camera notification link taking you to the debug page. It shows all the camera stats. The low storage, there isn’t anywhere it really can take you link wise, so that would just be purely letting you know I think.. |
Went ahead and added some tests for the notifications endpoint to make sure the values are what they should be. Could be expanded in another PR to test more endpoints. |
29ffe98
to
44a68f3
Compare
44a68f3
to
93ae8b0
Compare
Going to go ahead and close this, I think there will be a much better way to do this after the recent changes coming in 0.12 |
Any hints on the coming changes? |
No need for hints you can read about it here #4055 |
Idea
Per #3185 (comment)
It is a good idea for frigate to be able to post certain notifications.
Supported Notification Types (curently)
If anyone has any ideas or thoughts please let me know 👍