Skip to content

Conversation

leoAshu
Copy link
Owner

@leoAshu leoAshu commented Aug 24, 2025

🚀 Summary

This PR refactors the Verify flow to improve user experience by introducing a custom NotificationModal for success feedback.

Instead of relying on a small Toast, users now receive a clear, interactable and consistent success notification.

Key updates:

  • Designed a custom NotificationModal using the ModalView component.
  • Integrated NotificationModal into the Verify flow.
  • Replaced prior Toast based success handling with modal-based confirmation.
  • Aligns with previous PRs enhancing KYC and OTP verification flows, ensuring a consistent and unified feedback pattern.

🧩 Related Issues / Tasks

N/A

🛠️ Type of Change

  • 🐞 Bug fix
  • ✨ New feature
  • 🧨 Breaking change
  • 🔨 Enhancement/Refactor
  • 📝 Documentation
  • 📜 Other (please describe):

✅ Checklist

  • The code is formatted with Prettier (npm run format or npx prettier --write .)
  • PR references a linked Issue or Task number
  • Tested on both iOS and Android devices/emulators
  • No unnecessary logs or commented-out code
  • UI changes, if any, have been reviewed visually
  • All tests are passing (if applicable)
  • Added or updated documentation where necessary

📸 Screenshots / Videos (UI changes only)

N/A

🧪 Test Plan

  • Verified success modal displays after completing Verify flow.
  • Tested across OTP verification, KYC checks, and phone/email verification paths.
  • Ensured modal dismisses properly and no regressions occur in navigation.
  • Confirmed consistency in both light and dark modes.

@leoAshu leoAshu requested a review from mano-ranjan August 24, 2025 04:00
@leoAshu leoAshu self-assigned this Aug 24, 2025
@leoAshu leoAshu added enhancement Refactor style or logic feature New feature or request labels Aug 24, 2025
@leoAshu leoAshu merged commit cdefc9b into main Aug 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Refactor style or logic feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants