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.

Sign up for GitHub

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

Jump to bottom

Async notifications #180

Open
devagrawal09 opened this issue Jun 6, 2022 · 0 comments
Open

Async notifications #180

devagrawal09 opened this issue Jun 6, 2022 · 0 comments
Assignees
@devagrawal09
Labels
area: notifications sending push and email notifications to members for certain backend triggers backend involves work on the backend

Comments

@devagrawal09
Copy link
Collaborator

Currently the notification services are called synchronously when an action happens on the server. This should be asynchronous - the actions should just send a message to some service declaring what action has happened, then the notification service should pick that up at a later point and send the notifications. Currently notifications block the actual action, and if they fail, the whole action results in a failure, which is a big UX issue.
@devagrawal09 devagrawal09 added backend involves work on the backend area: notifications sending push and email notifications to members for certain backend triggers labels Jun 6, 2022
@devagrawal09 devagrawal09 self-assigned this Jun 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@devagrawal09 devagrawal09

Labels
area: notifications sending push and email notifications to members for certain backend triggers backend involves work on the backend
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@devagrawal09