Skip to content

Add complete authentication system with protected routes#65

Open
jikrana1 wants to merge 5 commits into
niharika-mente:mainfrom
jikrana1:feat/auth-login-signup
Open

Add complete authentication system with protected routes#65
jikrana1 wants to merge 5 commits into
niharika-mente:mainfrom
jikrana1:feat/auth-login-signup

Conversation

@jikrana1
Copy link
Copy Markdown

@jikrana1 jikrana1 commented Jun 3, 2026

📌 Related Issue

Fixes: #7

🔍 Describe your changes?

  • Added user registration functionality
  • Added user login with JWT authentication
  • Added logout functionality
  • Implemented protected routes for authenticated users
  • Added authentication state handling using localStorage
  • Updated navbar with conditional rendering based on authentication status
  • Displayed Create Product button only for logged-in users
  • Added premium UI for Login, Signup, and Navbar components
  • Improved user experience with redirects and authentication checks

📸 Screenshot

image image

🧪 Checklist

  • I have tested my changes locally
  • Authentication works correctly
  • Protected routes are functioning
  • No existing functionality was broken
  • Code follows project structure and standards

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

Someone is attempting to deploy a commit to the niharika-mente's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Aamod007 Aamod007 added enhancement New feature or request backend For changes related to the Express server, API routes, or MongoDB models frontend For any changes in the React UI, components, or Vite setup labels Jun 3, 2026
@Aamod007 Aamod007 self-requested a review June 3, 2026 09:29
@rishima17
Copy link
Copy Markdown
Collaborator

@jikrana1 Please resolve the conflicts.

Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there! Implementing a complete authentication system is a massive and important feature. Unfortunately, there are merge conflicts right now. Please rebase or pull the latest changes from main to resolve them, and I'll be happy to take a deeper look!

@Aamod007 Aamod007 added the security For issues related to vulnerabilities (like updating an out-of-date npm package) label Jun 4, 2026
Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Recent merges to the main branch have caused some merge conflicts here. Could you please pull the latest changes from main, resolve the conflicts, and update this PR? Thank you!

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

Labels

backend For changes related to the Express server, API routes, or MongoDB models enhancement New feature or request frontend For any changes in the React UI, components, or Vite setup Hard security For issues related to vulnerabilities (like updating an out-of-date npm package) SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Add Login & Signup authentication system (UI + Backend)

3 participants