-
Notifications
You must be signed in to change notification settings - Fork 29
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
Course Suggestions #24
Comments
Hey, @iampavangandhi Please check Below the mentioned Automated Bots Below.
|
Hey, @howdyAnkit Thanks for the suggestions we don't need these automated bots in TheNodeCourse Project because I and @vectorrb are the only contributors. We will surely use these bots in TradeByte as it's a team project. |
Hey @iampavangandhi I have a course suggestion of having a tutorial of bcrypt in TheNodeCourse as its widely used in hashing the credentials and then using it in storing the data in MongoDB. I would like to work on this as I'm pretty familiar with NodeJs. |
@ChiragBolakani Yes, you can try. you also have to create a Note: You don't have to do any linking. I will do that myself after merging the PR |
@iampavangandhi I have started working on it. |
@iampavangandhi I have completed the task. This is my pull request #38 |
@ChiragBolakani PR Merged 🎉 Welcome to TheNodeCourse 👍🏻 |
@iampavangandhi Please add the hacktoberfest label again to the repo. My pull request was not counted as the repo didn't had hacktoberfest label. The hacktoberfest also has a 14 days review period and the pull request needs to pass that. You may remove the hacktoberfest label after 10 November. Thank you. |
Hello @everyone 👋,
You can tell your thoughts and suggestions related to TheNodeCourse here.
Pavan Gandhi (Mentor)
The text was updated successfully, but these errors were encountered: