Skip to content
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

Solved issue 304 #315

Merged

Conversation

Saksham2k3s
Copy link
Contributor

Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.

This update restricts access to the Home and About pages, ensuring that only logged-in users can access these pages. The implementation leverages the Context API to manage authentication state and conditionally renders components based on the user's authentication status.

Fixes #304

Type of change

Please give a X on it which is applicable

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor Code
  • A documentation update
  • Others(mentioned in the issue number)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Test A Describe here

  • Attempted to access the Home and About pages without logging in.
  • Verified that the user is redirected to the login page when not authenticated.

Test B Describe here (if Requred)

  • Logged in with valid credentials.
  • Verified that the user can access the Home and About pages when authenticated.

Screenshorts and Vedios:

give screenshorts and vedio of the changes you made

Checklist:

give a X on it which is applicable

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Copy link

vercel bot commented Jun 13, 2024

@Saksham2k3s is attempting to deploy a commit to the vansh69's projects Team on Vercel.

A member of the Team first needs to authorize it.

@hustlerZzZ hustlerZzZ added gssoc This label is attached to the issues related to Gssoc'24 level2 labels Jun 15, 2024
Copy link
Collaborator

@hustlerZzZ hustlerZzZ left a comment

Choose a reason for hiding this comment

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

Great work

@hustlerZzZ hustlerZzZ merged commit e2c6c73 into VanshKing30:main Jun 15, 2024
0 of 2 checks passed
Copy link

Congratulations, Your pull request has been successfully merged 🥳🎉 Thank you for your contribution to the project 🚀 Keep Contributing!! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc This label is attached to the issues related to Gssoc'24 level2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Restrict Access to Home and About Pages to Logged-In Users Only
2 participants