-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Book travel - Book travel animation becomes blank while RHP is dismissed #49744
Comments
Triggered auto assignment to @MonilBhavsar ( |
👋 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:
|
don't know what caused it, but def NOT A BLOCKER |
This happens with all the animations, like on the wallet, security, and troubleshoot pages. |
let's close 😄 |
Thanks for looking! Demoting from blocker, but let's try if we can fix it |
ProposalPlease re-state the problem that we are trying to solve in this issue.Default name in name field displays user/report ID instead of user/report name What is the root cause of that problem?This happens as we render an empty view here if the user has navigated away. The issue we set hasNavigatedAway quickly before the RHP animation has been completed. This then shows the user an empty view while RHP is dismissing. What changes do you think we should make in order to solve the problem?We will change the state change here: to run after interactions, which will make it wait for the animation to be completed: App/src/components/Lottie/index.tsx Lines 82 to 88 in a4a344b
InteractionManager.runAfterInteractions(() => {
setHasNavigatedAway(true);
}); ResultScreen.Recording.2024-09-26.at.2.36.22.AM.mov |
@MonilBhavsar @neonbhai the reasons mentioned here suggest that we should do nothing. Or it may cause issues 1, 2, 3 But I guess that is only the case when the image hasn't loaded App/src/components/Lottie/index.tsx Lines 82 to 88 in a4a344b
|
Since this was reported as a regression, may be it comes from #48040 🤔 |
Not reproducible after reverting #48040 |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.40-0
Reproducible in staging?: Y
Reproducible in production?: 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): applausetester+kh230901@applause.expensifail.com
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
Book travel animation will not become blank while RHP is dismissed
Actual Result:
Book travel animation becomes blank while RHP is dismissed
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6615160_1727291080522.20240926_020845.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: