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

feat(hackathon): automated email setup #87

Closed
wants to merge 2 commits into from

Conversation

cloverzer0
Copy link
Contributor

An automated email subscription system.

The core functionality is implemented using AWS SES (Simple Email Service) and provides an API endpoint for subscribing to a newsletter.

  1. AWS SES Configuration: Configured AWS SES with credentials and region settings sourced from environment variables.
  2. subscribeToNewsletterRouter: Added a new TRPC procedure to handle newsletter subscriptions. This procedure:

Please review the changes and let me know if there are any adjustments or additional tests required.

Copy link

netlify bot commented Sep 3, 2024

Deploy Preview for cuhacking-portal-dev failed. Why did it fail? →

Name Link
🔨 Latest commit cc09883
🔍 Latest deploy log https://app.netlify.com/sites/cuhacking-portal-dev/deploys/66d7913ca3dbac0008767a76

@JeremyFriesenGitHub JeremyFriesenGitHub changed the title Muktar/feat/19 automated email setup feat(hackathon): automated email setup Sep 3, 2024
@JeremyFriesenGitHub JeremyFriesenGitHub added the enhancement New feature or request label Sep 3, 2024
@JeremyFriesenGitHub JeremyFriesenGitHub linked an issue Sep 3, 2024 that may be closed by this pull request
@JeremyFriesenGitHub
Copy link
Contributor

i haven't looked really into the code yet, but I'm wondering if the application form procedure commit in this PR should be in your other PR for the application form procedure?

@cloverzer0
Copy link
Contributor Author

i haven't looked really into the code yet, but I'm wondering if the application form procedure commit in this PR should be in your other PR for the application form procedure?

I made a branch from the first PR, thought it would be better to do that to be able to look at just one PR, this one.

@HasithDeAlwis
Copy link
Collaborator

@cloverzer0 we have to start phasing out the /hackathon directory, let's focus on getting this ready to be merged in the correct way. What is your availability this weekend to talk about this, let's prioritize this BEFORE we do anything more with the webhooks

@MFarabi619
Copy link
Member

MFarabi619 commented Oct 11, 2024

Is the development team budget finalized?

What are our estimates for how much this is going to cost for testing and prod?

When are mass emails being sent and what are the templates?

@MFarabi619
Copy link
Member

Closing as stale

@MFarabi619 MFarabi619 closed this Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Infra] Automated Email Setup
4 participants