Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Notifications: allow immediate dispatch #528

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

Open
johnstcn opened this issue Mar 25, 2025 · 0 comments
Open

Notifications: allow immediate dispatch #528

johnstcn opened this issue Mar 25, 2025 · 0 comments
Assignees

Comments

@johnstcn
Copy link
Member

Problem:

coder/coder#17091 adds functionality for sending push notifications.

This is currently not well-integrated into the rest of the nofications system.
The current notifications system has a enqueue system, and the current use-case for push notification is for immediate delivery. The current enqueue-dispatch logic takes too long for these notifications to be effective and timely.

Ideally we would have a way to immediately dispatch a notification without enqueuing it. In this case, delivery could not be guaranteed, but that's the tradeoff.

Proposed solution:

Add the capability to enqueue a notification for immediate dispatch, bypassing the database entirely. If we are not able to immediately dispatch, simply return an error to the caller.

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