Skip to content

Conversation

hariketsheth
Copy link

@hariketsheth hariketsheth commented Aug 26, 2023

Description

This pull request addresses Issue #1 , which proposes the addition of User Authentication and Registration Functionality to the project. The following tasks are completed as a part of the same

  • Implemented Passport.js and LocalStrategy for Authentication
  • Integrated bcrypt for secure Password Hashing
  • Connected User Model to store user information
  • Added Update User Profile Feature
  • Added Validation to Register Route

Related Issue

Fixes #1

Screenshots

image

image

image

Checklist

  • Code follows the project's coding guidelines
  • New code is adequately documented
  • No sensitive data is exposed in the logs or code
  • README and documentation have been updated (if necessary)
  • Ensuring No Merge Conflicts will happen upon merging the PR

@srummanf If you could please review this PR. 🎉
Thank you !

@srummanf
Copy link
Owner

Hello Bhaiyya, Can you check server>routes>admin.js file, I have already added that functionality. You can make these changes at that file.

@hariketsheth
Copy link
Author

Hello Bhaiyya, Can you check server>routes>admin.js file, I have already added that functionality. You can make these changes at that file.

Hey Rumman,
I missed that file out. I have removed the redundant routes.
Additionally, I have added validation to the register route and added Profile Updation

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add Authentication and User Registration

2 participants