Skip to content

Refactor identity page#923

Merged
zaelgohary merged 10 commits into
developmentfrom
development_refactor_identity
Feb 24, 2025
Merged

Refactor identity page#923
zaelgohary merged 10 commits into
developmentfrom
development_refactor_identity

Conversation

@zaelgohary
Copy link
Copy Markdown
Contributor

@zaelgohary zaelgohary commented Feb 16, 2025

Changes

  • Refactor identity
  • Remove spending & unnecessary nested widgets
  • Fix resend button
  • Add click effect to email & phone tiles
  • Show loading dialogue while verifying
  • Remove the sequential verification steps (email first, then mobile). Now, email verification is separate from mobile verification.
  • Combine verified & current phase widget in one
  • Remove unverified widget
  • Remove KYC-related code
  • Remove 'Verify now' dialogue

Related Issues

Tested Scenarios

  • Navigate to Identity multiple times and check user data
  • Logout then login w diff account and navigate to identity
  • Verify mobile
  • Verify mail
  • Try to verify old mobile again
  • Verify mail after moile is verified
  • Verify from icon
  • Verify from button
  • Edit mobile
  • Edit mail

@zaelgohary zaelgohary changed the title Refactor identity, remove spending & unnecessary nested widgets, fix resend btn Refactor identity page Feb 17, 2025
@zaelgohary zaelgohary marked this pull request as ready for review February 17, 2025 11:35
@zaelgohary zaelgohary marked this pull request as draft February 18, 2025 07:55
@zaelgohary zaelgohary marked this pull request as ready for review February 19, 2025 14:37
Copy link
Copy Markdown
Contributor

@AlaaElattar AlaaElattar left a comment

Choose a reason for hiding this comment

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

  • Here i'm not changing the phone number.

@AlaaElattar
Copy link
Copy Markdown
Contributor

  • Here i'm not changing the phone number.

  • I closed the dialog then clicked on the button again, the counter starts without even having a phone number.

@zaelgohary
Copy link
Copy Markdown
Contributor Author

Here i'm not changing the phone number.

Handled.

@AlaaElattar
Copy link
Copy Markdown
Contributor

  • I can't edit my phone number.

@zaelgohary
Copy link
Copy Markdown
Contributor Author

  • I can't edit my phone number.

What is the case or error you're facing? I've just edited and verified my phone now.

@AlaaElattar
Copy link
Copy Markdown
Contributor

  • I can't edit my phone number.

What is the case or error you're facing? I've just edited and verified my phone now.

I tested it again now and its working fine

Copy link
Copy Markdown
Contributor

@AlaaElattar AlaaElattar left a comment

Choose a reason for hiding this comment

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

Great Job ya zuzu 🚀

@zaelgohary zaelgohary merged commit b46b76c into development Feb 24, 2025
@zaelgohary zaelgohary deleted the development_refactor_identity branch February 24, 2025 10:57
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.

2 participants