Skip to content

Added protected routes for employer panel#229

Closed
Rovina05 wants to merge 2 commits intomainfrom
rovina-protected-routes
Closed

Added protected routes for employer panel#229
Rovina05 wants to merge 2 commits intomainfrom
rovina-protected-routes

Conversation

@Rovina05
Copy link
Copy Markdown
Collaborator

Implemented a ProtectedRoutes component for the employer panel using React Router.

Changes made:

  • Added ProtectedRoutes.jsx to handle authentication-based route protection
  • Updated App.js to wrap protected employer routes
  • Redirects unauthenticated users to the login page

Tested:

  • Verified protected pages redirect to login when not authenticated
  • Verified successful login flow and dashboard access after authentication

@neethuchandhavarkar
Copy link
Copy Markdown
Collaborator

This branch has merge conflicts. Could you please fetch the latest changes and raise the pull request again. Another thing to note is to add the screenshot in your pull request when you raise it.

@Rovina05
Copy link
Copy Markdown
Collaborator Author

Rovina05 commented Mar 30, 2026

Fetched latest changes from main, resolved conflicts, and added screenshots of the protected routes functionality. User won't be allowed to access any pages without logging in.

cant access any pages without log in

@neethuchandhavarkar
Copy link
Copy Markdown
Collaborator

Reviewed the pull request, this branch will delete the create shift file and also could you please elaborate the use of another files just for protected routes? Please raise the pull request again.

@neethuchandhavarkar
Copy link
Copy Markdown
Collaborator

Spoke over in teams and realized that this feature is already implemented in another PR which is merged in the main. Hence this is a duplicate PR. Closing this PR.

@neethuchandhavarkar neethuchandhavarkar deleted the rovina-protected-routes branch April 8, 2026 09:07
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.

2 participants