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

Added the Remember me button in authentication successfully #311

Merged
merged 2 commits into from
Jun 15, 2024
Merged

Added the Remember me button in authentication successfully #311

merged 2 commits into from
Jun 15, 2024

Conversation

codercake
Copy link
Contributor

This implementation includes:

  1. Centered "Forgot Password" link using flex justify-center.
  2. "Remember Me" checkbox with appropriate HTML elements.

No new dependencies are required for these changes.

Fixes #302

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?

Manual UI Inspection:

  1. Opened the application in a local development environment.
  2. Navigated to the login page.
  3. Visually inspected the layout to ensure the "Forgot Password" link is centered.
  4. Verified that the "Remember Me" checkbox appears correctly and is functional.

Checkbox Functionality Test:

  1. Checked the behavior of the "Remember Me" checkbox by selecting and deselecting it.
  2. Verified that the checkbox's state is correctly tracked.

Responsive Design Test:
Resized the browser window to various dimensions to ensure the changes work well on different screen sizes, including mobile, tablet, and desktop.

Screenshots and Videos:

Screenshot 2024-06-12 165322

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 12, 2024

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

A member of the Team first needs to authorize it.

@codercake
Copy link
Contributor Author

@VanshKing30 can you review the PR?

@hustlerZzZ
Copy link
Collaborator

@VanshKing30 can you review the PR?

Did you implement the backend logic as well?

@codercake
Copy link
Contributor Author

@VanshKing30 can you review the PR?

Did you implement the backend logic as well?

yes. i did.

@hustlerZzZ hustlerZzZ added gssoc This label is attached to the issues related to Gssoc'24 level1 labels Jun 15, 2024
@hustlerZzZ hustlerZzZ merged commit a709569 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 level1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: Add "Remember Me" Section for Authentication
2 participants