You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
coder/coder#17091 added a webpush package that supports sending notifications over WebPush.
This was not integrated into the rest of the notifications subsystem because its preferred mode of usage is more akin to UDP than the existing "TCP" reliable notification delivery system.
In order to integrate this properly, we'll need to do the following:
Add the ability to immediately dispatch a notification while skipping the enqueuing process. This should be a new code path and should not be configurable by end-users (i.e. "immediate dispatch" versus "retry dispatch" should be hard-coded by design)
Add the ability to dispatch a notification with an arbitrary message body. This should only be possible with arbitrary dispatch for now, as the current architecture for retryable dispatch requires a template to be stored in the database.
Add the webpush dispatcher as a "first-class" notification target.
Notes:
We'll have to be very careful about how we name things here so as not to conflate "webpush" notifications with the concept of a "notification" in coder/coder.
Tasks:
Allow immediately* dispatching a notification.
Add the ability to dispatch a notification with an arbitrary message body
Add the webpush notifier as a "first-class" notification target
* or as close to immediately as possible
The text was updated successfully, but these errors were encountered:
coder/coder#17091 added a webpush package that supports sending notifications over WebPush.
This was not integrated into the rest of the notifications subsystem because its preferred mode of usage is more akin to UDP than the existing "TCP" reliable notification delivery system.
In order to integrate this properly, we'll need to do the following:
Add the ability to immediately dispatch a notification while skipping the enqueuing process. This should be a new code path and should not be configurable by end-users (i.e. "immediate dispatch" versus "retry dispatch" should be hard-coded by design)
Add the ability to dispatch a notification with an arbitrary message body. This should only be possible with arbitrary dispatch for now, as the current architecture for retryable dispatch requires a template to be stored in the database.
Add the webpush dispatcher as a "first-class" notification target.
Notes:
We'll have to be very careful about how we name things here so as not to conflate "webpush" notifications with the concept of a "notification" in
coder/coder
.Tasks:
* or as close to immediately as possible
The text was updated successfully, but these errors were encountered: