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 formik to register page. Fixed issue #123 #124

Merged
merged 5 commits into from
Apr 12, 2023

Conversation

SANDESHSOBARAD
Copy link
Contributor

No description provided.

Copy link
Contributor

@priyankavrao97 priyankavrao97 left a comment

Choose a reason for hiding this comment

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

Working as expected.

@DevangRaval1
Copy link
Contributor

ReviewSandesh

Some problem with validation

@AbhinavReddy-Dev
Copy link
Contributor

@SANDESHSOBARAD Can we have a "view password" icon inside the password field, also the "confirm password" field which re-confirms the password set?

@SANDESHSOBARAD
Copy link
Contributor Author

Yes, I will work on that.

@hardikpatil
Copy link
Contributor

Looks good to me. Can you add the same view password icon inside the confirm password field

Screenshot 2023-04-11 at 17 18 58

@SANDESHSOBARAD
Copy link
Contributor Author

I think it doesn't need to be there, cause if the password field and confirm password field are not matching it won't let you sign up. Basically you can see what you typed in the password field and type the same in confirm password field to match.

Copy link
Contributor

@abhilashSreenivasa abhilashSreenivasa left a comment

Choose a reason for hiding this comment

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

Looks good to me as well.

@SANDESHSOBARAD SANDESHSOBARAD merged commit f5048c8 into frontend Apr 12, 2023
@SANDESHSOBARAD SANDESHSOBARAD deleted the frontend-registerformik branch April 12, 2023 00:46
@hardikpatil hardikpatil added the UI Concerns UI presentation label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI Concerns UI presentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants