-
Notifications
You must be signed in to change notification settings - Fork 0
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
Nodemailer Integration #49
Comments
So will the email we are sending to aka the admins email also be provided when making the request? |
The admin's email will be pre-configured but I still need to talk to them about it; for now, have the component send the email to |
This is my current thought process: And Do I have to use the current sign-in page or make my own sign in page, because the current one is only Email and password |
Creation of the sign in page is implemented by another team during this sprint, feel free to use hardcoded dummy values for now until we connect the sign in page to the nodemailer flow. |
and also when I try to create the transporter i get the error that: from the reddit thread, docs and speculation Im guessing we need to make the mail less secure to use it as a transporter I am kind of stuck on what to do, because the link also redirects to the something into a more "secure service" |
Help here would be appreciated we are kinda stuck 😅especially for testing, though the code is complete and "looks right" just needs a bit of review as far as i can tell, because we aren't sure why the error is showing up |
There should be an application password in the .env file on the drive, are you using that? There's two emails -- the one in this thread, and the one in the backend .env doc. |
I am using the one on the drive, yep! |
I just tested it and was able to send the email; there's a couple of changes that need to be made to do this. |
I was having a There's a bit of dependency hell involved in adding the |
I did those things, can you see what I am doing wrong in my branch? The problem is arriving at the transporter still for me 😔 |
im having the same issue as priyansh when login with google :( |
Instructions
We’re going to use nodeMailer to notify the admins of Center Stage when someone has created an account and is waiting for approval. In this task, you’re going to…
backend:
server/common
frontend:
Note:
inspect element
, be sure to selectiPhone SE
for test!!!Heads up:
client/src/components/playground/Playground.jsx
to test out anything before putting it in their respective pages, or if there currently is no respective pageCreate a temporary email template to send (this could be a simple component with some text).
Create a temporary component to test out the email functionality. Add format validation for the email input.
Use the
react-html-email
library to render the email template.Call the backend route to send the email.
Notes
.env
in the drive..env
is an App password to give nodeMailer access to the account, not the actual Google password.[email protected]
Center$tage!!
Acceptance Criteria
backend
server/routes/nodeMailer.js
frontend
Resources
backend:
frontend:
The text was updated successfully, but these errors were encountered: