-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Added the icon and title to the site #56
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for github-spy ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@parassareen4 : use a proper commit message. |
b2000a3
to
13e4a75
Compare
- Corrected passport authentication flow for login route. - Fixed CORS setup to handle credentials in frontend requests. - Implemented session handling for login and signup at (/signup)and(/login). - Updated API endpoints to return appropriate error messages and status codes
signup is on /signup route and login on /login |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the icon file name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything is good just try to write good code for backend
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any changes needed here?
@parassareen4 good all changes look fine to me, @mehul-m-prajapati please look into this PR. I think it's good enough to be merged now. |
@mehul-m-prajapati Kindly merge this PR |
@yashksaini-coder : okay, I will test and merge today. |
the tests went fine? |
Related Issue
🚀 Feature: Add login/signup page to connect with backend auth APIs of nodejs #19
Description
How Has This Been Tested?
Screenshots (if applicable)
Type of Change