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

LoginPage validation update #312

Closed
wants to merge 3 commits into from

Conversation

HardCoder404
Copy link

Description :

In this bug, I just added a validation in Login Page.

ScreenShot :

Screenshot 2024-05-13 201921
Screenshot 2024-05-13 201937

@HardCoder404
Copy link
Author

Please preview the Changes and let me know.

@rishicds
Copy link
Collaborator

Hi @HardCoder404 , please fix the merge conflicts, it will be merged after that, also mention which issue this closes.

@HardCoder404
Copy link
Author

The conflict has been resolved. Please review and merged it.

Copy link
Collaborator

@codewarnab codewarnab left a comment

Choose a reason for hiding this comment

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

sorry to bother you again please move the validation inside the login function which is in the userAuthService.js it will maintain the code quality .
also their is already email validatory so no need to valid it two times

login(email, password, setError, setButtonState);
};

=======
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove this thing also

@codewarnab
Copy link
Collaborator

@HardCoder404 its been 10 days give update or we have to close the pr

@codewarnab codewarnab closed this May 29, 2024
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.

3 participants