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

Skip to content

Improve notifier part #247

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

Closed
defelmnq opened this issue Dec 10, 2024 · 0 comments · Fixed by coder/coder#16250
Closed

Improve notifier part #247

defelmnq opened this issue Dec 10, 2024 · 0 comments · Fixed by coder/coder#16250
Assignees

Comments

@defelmnq
Copy link

In order for the OOM and OOD notifications to work - we need to change a bit the notifiers :

  • Create a new migration to add the template for the new events. It is a simple DB migration.
  • Modify both the FE and BE to add a new event with the preferences.
    • It should be done in the admin side to let admins choose if the notifications should be sent through webhooks or email
    • It should be done in the user side to let users turn it on or off.

⚠ This component is the last one that should be merged - once merged the connection between the processing part and the notifiers can be done.

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

Successfully merging a pull request may close this issue.

3 participants