-
Notifications
You must be signed in to change notification settings - Fork 5
Push notifications for new tags and imported folders #148
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
base: main
Are you sure you want to change the base?
Conversation
Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry. |
This would be super useful to me! But, would it be possible to also allow configuring a webhook URL for notifications on the server-side? This would allow me to instead forward notifications to e.g. a ntfy server or Discord channel. |
I'm not currently using ntfy myself, and from what I’ve seen, the standard Web Push + Notification API seems a lot more powerful, at least for enabling interactions directly within our app. In general, I lean toward supporting open standards first, since they offer the broadest compatibility. But once that’s solid, adapting it to send messages to alternative services could definitely be doable. That said, it should be relatively straightforward to patch the server-side notification function to forward messages to additional endpoints, assuming some kind of configuration is enabled. I won’t prioritize support for external endpoints right now, but I’ll keep it in mind, especially when considering the database representation of push subscribers. |
0772df0
to
b42ebdc
Compare
…rated frontend types.
components out of debug page.
webhooks and push notifications (wip).
version is detected.
This PR introduces the foundational functionality for sending and receiving push notifications to registered devices. It includes everything necessary to implement basic notification flows, including sending from the backend and displaying on the client side.
This is a draft! And we still have to implement quite some things to get this working.
TODOs:
Technicals