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.

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

S25 Sprint 1- Add notification upon signup feature #430

Open
3 tasks
johannesq23 opened this issue Jan 22, 2025 · 2 comments · May be fixed by #432
Open
3 tasks

S25 Sprint 1- Add notification upon signup feature #430

johannesq23 opened this issue Jan 22, 2025 · 2 comments · May be fixed by #432
Assignees

Comments

@johannesq23
Copy link
Contributor

johannesq23 commented Jan 22, 2025

Context

Some admins would like to be notified upon signup or request to signup for an event. We want to send a notification email to the admins once someone registers or submits a request to join an event.

Requirements

We want the email feature to be optional, so add a toggle button here, which says "notify admins upon registration." You should modify the eventparent schema with a notifyadmin field to determine wether or not to notify the admins

Image

Explore src/pages/api/registrations/index.ts and src/utils/mailersend-email.js as a start. Sorry this is a rushed ticket- the last ticket I wrote was already a PR. Reach out to me if you are confused or have any questions

Acceptance Criteria

  • Email sends upon registration
  • Schema updated correctly
  • Previous function not impacted
@ylw-PotatoTornado
Copy link

@johannesq23 Hi Johannes, in terms of admin(s) to be notified upon event registration, would you please clarify if:

1- only the event creator admin (i.e. the "eventContactEmail" in EventParent entity)
2- all admins within the org

need to be notified?

@johannesq23
Copy link
Contributor Author

@ylw-PotatoTornado only the event creator admin needs to be notified- thanks

@ylw-PotatoTornado ylw-PotatoTornado linked a pull request Jan 29, 2025 that will close this issue
3 tasks
@ylw-PotatoTornado ylw-PotatoTornado linked a pull request Jan 31, 2025 that will close this issue
3 tasks
@ylw-PotatoTornado ylw-PotatoTornado linked a pull request Jan 31, 2025 that will close this issue
3 tasks
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.

2 participants