-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Android & iOS hybrid - Login - "Welcome to Expensify" modal is missing video #60448
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
Comments
Triggered auto assignment to @jasperhuangg ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
I'm not able to get this video to show up at all. Will check the preconditions. |
Hmm so this is the logic for showing this modal... App/src/hooks/useOnboardingFlow.ts Lines 37 to 70 in 10fbf03
Lots of conditions there. I am not too sure what state an account needs to be in to get this modal to appear. |
Was able to repro with expensifail.com account. There's lots of logs but this sticks out to me:
I think that's coming from ExpoAV? This PR seems like a potential culprit: #59896 |
@JakubKorytko @mountiny @DylanDylann Can you look at this please? |
Ok this was somehow caused by #60078 I can't tell exactly how so going to revert. |
We CP'd a fix for this. Asked for a re-test. |
@marcaaron bug is not reproducible 2025-04-18.8.01.58.AM.mp4 |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.1.29-5
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:
Expected Result:
"Welcome to Expensify" modal will show the video
Actual Result:
"Welcome to Expensify" modal is missing video
Workaround:
Unknown
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:Screenshots/Videos
Add any screenshot/video evidence
Bug6805283_1744908253206.Screen_Recording_20250418_004043_Expensify.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: