Skip to content

feat: account notifications #17

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
Closed
3 of 5 tasks
dannykopping opened this issue Jul 18, 2024 · 1 comment · Fixed by coder/coder#14367
Closed
3 of 5 tasks

feat: account notifications #17

dannykopping opened this issue Jul 18, 2024 · 1 comment · Fixed by coder/coder#14367
Assignees
@dannykopping @mtojek
Labels
project/notifications

Comments

@dannykopping
Copy link
Collaborator

dannykopping commented Jul 18, 2024 •
edited
Loading

Relates to coder/coder#13328

Create notification_template instances for the following events, and enqueue notifications with those templates:

These two notifications require some additional functionality, and perhaps require RFCs:
@dannykopping dannykopping changed the title Implement account notifications Account notifications Jul 18, 2024
@dannykopping dannykopping changed the title Account notifications feat: account notifications Jul 18, 2024
@dannykopping dannykopping mentioned this issue Jul 18, 2024
Closed
6 tasks
@mtojek mtojek assigned dannykopping and mtojek and unassigned dannykopping Jul 19, 2024
@mtojek mtojek mentioned this issue Jul 25, 2024
Merged
This was referenced Aug 2, 2024
Merged
Closed
Closed
@mtojek
Copy link
Member

mtojek commented Aug 2, 2024

Hey @dannykopping, I'm done with notifications before holidays/PTO.

  1. I extracted more complex items to separate issues, agree they may require mini RFCs.
  2. I assigned this issue to you to either implement user account suspended/reactivated or re-assign to somebody else
  3. Do you think we can update the listing in Create notification templates for key events coder#13328? We have implemented or declined a few items.

@mtojek mtojek mentioned this issue Aug 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dannykopping dannykopping

@mtojek mtojek

Labels
project/notifications
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add notification for suspended/activated account coder/coder
2 participants
@dannykopping @mtojek