-
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
[HOLD for payment 2024-08-01] [$250] Settings - Left hand menu and RHP do not open instantly when opening from link #44585
Comments
Triggered auto assignment to @marcaaron ( |
👋 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:
|
@marcaaron FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
We think that this bug might be related to #vip-vsp |
Triggered auto assignment to @MitchExpensify ( |
Job added to Upwork: https://www.upwork.com/jobs/~011fca7dda4a362b49 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @abdulrahuman5196 ( |
@marcaaron, @MitchExpensify, @abdulrahuman5196 Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Waiting on proposals, Melvin |
Please re-state the problem that we are trying to solve in this issue.Settings - Left hand menu and RHP do not open instantly when opening from link What is the root cause of that problem?The root cause of the issue is that the necessary data for rendering the left-hand menu and right-hand panel (RHP) is not immediately available when navigating directly to a specific URL, causing a delay in rendering and resulting in incomplete UI display. This occurs because the components try to render before the data is fully loaded, leading to a delay in the UI being displayed and the selected tab not being highlighted. What changes do you think we should make in order to solve the problem?To resolve this issue, ensure that the component explicitly handles the loading state and waits for the necessary data before rendering the main content. Here’s how you can do it:
video:- https://drive.google.com/file/d/1OP8jCA7-NAS8CtGB8jPgdhnhJKyKhS9q/view?usp=sharing |
Please re-state the problem that we are trying to solve in this issue.When opening the settings from link, the left-hand menu and RHP do not open instantly. What is the root cause of that problem?The What changes do you think we should make in order to solve the problem?I initially tried to control the Lottie animation manually as recommended in the documentation. However, this approach did not work as expected. useEffect(() => {
// Attempt to control the animation programmatically instead of using autoPlay
// animation.current?.play();
}, []); To avoid the infinite re-rendering, I used a state variable autoPlayDelay that temporarily holds the autoPlay value. This conditionally renders an empty view which prevents the infinite re-renders. const [autoPlayDelay, setAutoPlayDelay] = useState(autoPlay);
useEffect(() => {
const timeout = setTimeout(() => {
autoPlay && setAutoPlayDelay(false);
}, 0);
return () => clearTimeout(timeout);
}, []);
// If the image fails to load or app is in background state, we'll just render an empty view
// using the fallback in case of a Lottie error or appState.isBackground to prevent
// memory leak, see issue: https://github.com/Expensify/App/issues/36645
//
// autoPlayDelay is using to prevent infinite re-renders when autoPlay is enabled
if (isError || appState.isBackground || autoPlayDelay) {
return <View style={[aspectRatioStyle, props.style]} />;
} Visual representation here |
Triggered auto assignment to @RachCHopkins ( |
We are looking for proposals |
Hey! I’m Wojtek from Software Mansion, an expert agency, and I’d like to work on this issue. |
Sounds good! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.11-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-08-01. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
|
@RachCHopkins, @marcaaron, @WojtekBoman, @ishpaul777 Huh... This is 4 days overdue. Who can take care of this? |
I think we're just waiting on payment for @ishpaul777 at this point. |
Payment approved in UpWork for @ishpaul777 - sorry for the delay Ishpaul! |
Payment Summary:
|
Contributor has been paid, the contract has been completed, and the Upwork post has been closed. |
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.3-1
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
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
In Step 1, when opening the settings from link, the left-hand menu and RHP will open instantly
In Step 5, the selected tab will be highlighted
Actual Result:
In Step 1, when opening the settings from link, the left-hand menu and RHP do not open instantly
In Step 5, the selected tab is not highlighted
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6526859_1719530885004.20240628_064625.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @RachCHopkinsThe text was updated successfully, but these errors were encountered: