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

Mkaleem neslit mobile onboarding #10518

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KaleemNeslit
Copy link
Collaborator

@KaleemNeslit KaleemNeslit commented Jan 8, 2025

Link to Issue

Closes: #10247

Description of Changes

  • Adds FIXME widget to TODO page.
  • updated the ui for welcome modal
  • added the animation for user onboarding
  • create the new modal for optional wallet connect
  • created the new modal for Notification to trigger permission for mobile side
  • created the referral modal
  • made the modal full screen for mobile

"How We Fixed It"

  • Adds FIXME widget to TODO page.
  • updated the ui for welcome modal
  • added the animation for user onboarding
  • create the new modal for optional wallet connect
  • created the new modal for Notification to trigger permission for mobile side
  • created the referral modal
  • made the modal full screen for mobile

Test Plan

  • currently you cannot test it directly because to completing the flow we need to complete auth in mobile and sent the data for first time user to web . so for testing what you can do it goto WelcomeOnboardModal and const isFromWebView = Boolean(window?.ReactNativeWebView); set this true also changed the isOpen to true as well .
    make sure you are already signIn . currently we are not checking for authentication type user used to login/ create the account .

for seeing the animation navigate to this url http://localhost:8080/onboarding .

@masvelio
Copy link
Contributor

masvelio commented Jan 9, 2025

make sure myself or/and @mzparacha are requested for review when PR is ready

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