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

Backend Routes: Directory Approval/Denial Email #58

Open
Miyuki-L opened this issue Feb 4, 2025 · 3 comments · May be fixed by #76
Open

Backend Routes: Directory Approval/Denial Email #58

Miyuki-L opened this issue Feb 4, 2025 · 3 comments · May be fixed by #76
Assignees
@raymondwusuper
Labels
Backend Manage Counselors

Comments

@Miyuki-L
Copy link
Member

Miyuki-L commented Feb 4, 2025

Flow:

  • Frontend Approval of Directory --> backend route --> sends email to member that is approved that they have been approved
  • Frontend Deny with reasoninig --> backend --> Send email to denied member with reasoning for denial

Pap Example of Email:
https://github.com/TritonSE/PAP-Inventory-Processing/blob/ea2e7275d40f10fca354eb749f253a52cc75ef60/backend/src/services/emails.ts#L79

Packages: {Nodemailer](https://www.nodemailer.com/)

Make Sure Credentials Go in ENV File
@raymondwusuper
Copy link
Contributor

completed, make sure it works on your end

Image Image

@raymondwusuper
Copy link
Contributor

might be some issues because someone changed npm start to nodemon src/app.ts so change that to index or add another macro to test email routes

@raymondwusuper raymondwusuper mentioned this issue Feb 26, 2025
Closed
@raymondwusuper raymondwusuper linked a pull request Feb 26, 2025 that will close this issue
58 #76
Open
@raymondwusuper raymondwusuper linked a pull request Feb 26, 2025 that will close this issue
Open
@Miyuki-L
Copy link
Member Author

might be some issues because someone changed npm start to nodemon src/app.ts so change that to index or add another macro to test email routes

Some miss communication, we are movign things in the index over to the src/app.ts file and have it start both the express server and the mongo db connection

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raymondwusuper raymondwusuper

Labels
Backend Manage Counselors
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

58 TritonSE/SPLAGen-Website
2 participants
@raymondwusuper @Miyuki-L