-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
GH-3652 Add forgot password on sign-in page #3789
GH-3652 Add forgot password on sign-in page #3789
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @i-am-chitti, looks good from a high level code perspective. I have added comments.
packages/twenty-front/src/modules/auth/sign-in-up/components/SignInUpForm.tsx
Outdated
Show resolved
Hide resolved
packages/twenty-front/src/modules/auth/sign-in-up/components/SignInUpForm.tsx
Outdated
Show resolved
Hide resolved
Hi @i-am-chitti! Any update on this one? :) |
Sorry @charlesBochet, I didn't get to work on this past days. I'll work on this today. |
@charlesBochet Please check the PR feedback comments for new reply. |
@martmull could you do the split? :) |
@i-am-chitti @charlesBochet I will |
ff799f8
to
c056498
Compare
Hey @i-am-chitti this is merged. Thank you very much for your contribution it is really good work. Cheers |
Description
This PR adds a forgot your password button on sign-in page.
Issue
Screenshots
Technical choices
Note
graphql.tsx
is containing some irrelevant changes. Not sure, how they are popping into though had tried with merging latest main branch at the moment. Feel free to push to update the generated graphql at your will.