Send email when user sign up using node-mailer about bunk-manager-mern

[udaymittal7]

Hey, can I take this task?

[Mustafiz04]

Hey!
Yes go for it.

[ADRE9]

Hold Your PRs for somedays until we organize a meet and explain the repos to you

[udaymittal7]

Yeah sure.

[udaymittal7]

I have implemented the code for this. Should I review it with you before making a pull request?

[Mustafiz04]

Have you run it in your local host??
Is it run perfectly??

If yes make a PR with proper name and with new branch.
We will receive it.

[udaymittal7]

The changes I am making in the code after running on vs code and then running the server is not reflected on the site running on my localhost. Not this email one specifically, but any change I make. What should I do?

Also I am getting this error after running npm run dev

[udaymittal7]

I have checked the implementation of my code in one of my project and it is working just fine. Should I make the PR and then you can review it?

[Mustafiz04]

Have you run npm install in your terminal??
See this is mongo error there are some syntax mistake.
Go through the line where MongooseError start.

[Mustafiz04]

First run successfully on your local then make PR.
It will be better for both of us.

[ADRE9]

Can you show @Mustafiz04 a live version before making a PR? If @Mustafiz04 agrees you can make a PR to the branch-1.

[udaymittal7]

Yes I did run npm install in both client folder and the main folder as mentioned by you. I think the process.env.URI is not defined which is causing the issue.

Should i connect it with my mongoDB to test it?

[udaymittal7]

@ADRE9 Yes, I will only make the PR on his confirmation. I also don't want to burden you by making the PR for reviewing it.

[udaymittal7]

My changes are still not be able to reflect when I run my code and start the server. Though the mongoose error is gone.
What to do @Mustafiz04

[udaymittal7]

Hey I made the PR for it and it got merged. Please check.