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

fix: remember me auto checks while login #1186

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

aniketxpawar
Copy link

Description

Please provide a brief description of the changes or additions made in this pull request.
The initial values of Form were not being passed to LoginForm from Login, I passed in the form prop

Related Issues

Fixes: #1177

If this pull request is related to any issue(s), please list them here.
#1177

Steps to Test

-Log out if already logged in
-Enter email password
-Uncheck Remember me checkbox
-Enter login
-The remember me checkbox wont auto check

Provide steps on how to test the changes introduced in this pull request.

Screenshots (if applicable)

If your changes include visual updates, it would be helpful to provide screenshots of the before and after.

Before:
image
image

After:
image

Checklist

  • [✅ ] I have tested these changes
  • [✅ ] I have updated the relevant documentation
  • [✅ ] I have commented my code, particularly in hard-to-understand areas
  • [✅ ] I have made corresponding changes to the codebase
  • [✅ ] My changes generate no new warnings or errors
  • [✅ ] The title of my pull request is clear and descriptive

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.

1 participant