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 OTP and Profile pages UI #706

Merged
merged 3 commits into from
Jun 27, 2024
Merged

Conversation

Asymtode712
Copy link
Contributor

Description

Fixes #682

  • Added UI for OTP and Profile
  • Added validators for both the pages

Video

For OTP:

Helpops-Hub.-.Personal.-.Microsoft.Edge.2024-06-28.02-13-01_Trim.mp4

For Profile:

Helpops-Hub.-.Personal.-.Microsoft.Edge.2024-06-28.02-15-26.mp4

Copy link

vercel bot commented Jun 27, 2024

@Asymtode712 is attempting to deploy a commit to the MD AZFAR ALAM's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@Asymtode712
Copy link
Contributor Author

@mdazfar2 Please review and merge this PR

Copy link

vercel bot commented Jun 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
helpops-hub2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 27, 2024 9:17pm

@mdazfar2
Copy link
Owner

@Asymtode712, I think there is no need for it because this page is for creating a new account after email verification. If the email is not verified, then how will the user set everything from here? It also takes up space.


image

@Asymtode712
Copy link
Contributor Author

@mdazfar2 Ok....fixing this now

@mdazfar2
Copy link
Owner

mdazfar2 commented Jun 27, 2024

@Asymtode712, after clicking on continue on the OTP page, it should open the username and password page. Also, add a small black arrow here so the user can go back to change the email.


image


Did you see this commit message? 💭

Copy link
Owner

@mdazfar2 mdazfar2 left a comment

Choose a reason for hiding this comment

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

@Asymtode712 Great work

  • I am merging it now. There are some changes left, you can do later.

@mdazfar2 mdazfar2 merged commit 5c6b0c2 into mdazfar2:main Jun 27, 2024
1 of 2 checks passed
@Asymtode712
Copy link
Contributor Author

@mdazfar2 Done with the changes:

  • refactored rendering of Profile

    Helpops-Hub.-.Personal.-.Microsoft.Edge.2024-06-28.02-44-02_Trim.mp4
  • added back button and its functionality in OTP:

Screenshot (81)

@Asymtode712
Copy link
Contributor Author

@mdazfar2 Done with all the changes which you requested.....please check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: Add UI for OTP verification and Forgot Password
2 participants