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

[$250] Blank page appears between authentication and loading UI #49934

Open
2 of 6 tasks
Julesssss opened this issue Sep 30, 2024 · 64 comments
Open
2 of 6 tasks

[$250] Blank page appears between authentication and loading UI #49934

Julesssss opened this issue Sep 30, 2024 · 64 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2

Comments

@Julesssss
Copy link
Contributor

Julesssss commented Sep 30, 2024

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.0.41-1 and earlier
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): Customer accounts with a medium-large amount of data

Action Performed:

  • Device is on a slow or restricted network
  • Launch New Expensify app
  • Log in with an account with many reports

Expected Result:

  • There will be no white screen after login

Actual Result:

  • White screen appears briefly after login

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

372109345-c082d55a-26d9-4ade-973d-e1562d5262f7.mov
371763733-127dec60-2e34-4b95-af30-0bae07ce10b2.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021840753836543659093
  • Upwork Job ID: 1840753836543659093
  • Last Price Increase: 2024-11-25
  • Automatic offers:
    • shubham1206agra | Reviewer | 105126183
    • NJ-2020 | Contributor | 105126185
Issue OwnerCurrent Issue Owner: @NJ-2020
@Julesssss Julesssss added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 30, 2024
@Julesssss Julesssss self-assigned this Sep 30, 2024
Copy link

melvin-bot bot commented Sep 30, 2024

Job added to Upwork: https://www.upwork.com/jobs/~021840753836543659093

@melvin-bot melvin-bot bot changed the title Blank page appears between authentication and loading UI [$250] Blank page appears between authentication and loading UI Sep 30, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 30, 2024
Copy link

melvin-bot bot commented Sep 30, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External)

Copy link

melvin-bot bot commented Sep 30, 2024

Triggered auto assignment to @OfstadC (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@Julesssss
Copy link
Contributor Author

Hey @blazejkustra, in case you are interested... I also added your repro video.

Copy link

melvin-bot bot commented Oct 3, 2024

@Julesssss, @OfstadC, @shubham1206agra Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Oct 3, 2024
Copy link

melvin-bot bot commented Oct 7, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Copy link

melvin-bot bot commented Oct 7, 2024

@Julesssss, @OfstadC, @shubham1206agra Still overdue 6 days?! Let's take care of this!

@Julesssss Julesssss removed their assignment Oct 8, 2024
@melvin-bot melvin-bot bot removed the Overdue label Oct 8, 2024
@Julesssss
Copy link
Contributor Author

We need good reproduction steps before making this open to contributors.

@OfstadC OfstadC added the Needs Reproduction Reproducible steps needed label Oct 8, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@mvtglobally
Copy link

@truph01
Copy link
Contributor

truph01 commented Oct 10, 2024

Cannot reproduce this issue after we migrate to useOnyx.

@Julesssss
Copy link
Contributor Author

@truph01, can you reproduce on web when throttling the network?

@truph01
Copy link
Contributor

truph01 commented Oct 10, 2024

@Julesssss No, I cannot reproduce in both web and ios after this PR is merged.

@Julesssss
Copy link
Contributor Author

Okay thank you. It looks like the PR was merged just after the latest test. Lets retry once more.

@Julesssss Julesssss added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Oct 10, 2024
@truph01
Copy link
Contributor

truph01 commented Oct 11, 2024

@Julesssss FYI, the above PR is reverted, so the bug can be reproduced.

@melvin-bot melvin-bot bot added the Overdue label Oct 11, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 9, 2024
@muttmuure muttmuure moved this to MEDIUM in [#whatsnext] #quality Dec 9, 2024
@shubham1206agra
Copy link
Contributor

@NJ-2020 Update please

@NJ-2020
Copy link
Contributor

NJ-2020 commented Dec 11, 2024

Hi, I am very sorry.
I am OOO now for few days.
i am currently mourning my father who has passed away yesterday.

Thanks

cc: @shubham1206agra

@melvin-bot melvin-bot bot removed the Overdue label Dec 11, 2024
Copy link

melvin-bot bot commented Dec 16, 2024

@robertjchen, @OfstadC, @shubham1206agra, @NJ-2020 Eep! 4 days overdue now. Issues have feelings too...

@melvin-bot melvin-bot bot added the Overdue label Dec 16, 2024
@shubham1206agra
Copy link
Contributor

Waiting for @NJ-2020 to come back.

@robertjchen robertjchen added Weekly KSv2 and removed Daily KSv2 labels Dec 18, 2024
@melvin-bot melvin-bot bot removed the Overdue label Dec 18, 2024
@robertjchen
Copy link
Contributor

My condolences, no rush 🙏

@NJ-2020
Copy link
Contributor

NJ-2020 commented Dec 19, 2024

Update:

  • Loading spinner (solution) doesn't work because If I add like loading spinner, the loading spinner will be displayed every time the AuthScreens loading like logout, opening the app

Solution
I think after we login before redirecting/displaying AuthScreens page we can wait until the AuthScreens component fully loaded and then we can add loading spinner in the login page when the AuthScreens page is still loading

cc: @shubham1206agra wdyt?

@shubham1206agra
Copy link
Contributor

I believe this can work. @NJ-2020 Do you have a POC for this?

@NJ-2020
Copy link
Contributor

NJ-2020 commented Dec 28, 2024

@shubham1206agra Not yet

After doing some digging into the code, I found:

  • Instead of using Suspense we can use like isBottomtabbarLoaded(not exist but if I am not mistaken this key is exist before) because if we use Suspense the blank screen still appear I think maybe due to some components inside the AuthScreens, this also to make sure that we will hide the loading spinner if the bottom tab bar has been rendered
  • And also I think to prevent the loading spinner from being displayed every time the AuthScreens component is loading, we can put the loading spinner around this if statement to make sure that we're displaying the loading spinner after we're authenticated(after login)
    if (authenticated) {
    // These are the protected screens and only accessible when an authToken is present
    return (

@NJ-2020
Copy link
Contributor

NJ-2020 commented Dec 30, 2024

@shubham1206agra Good news, It worked! But I still have an issue for android (native), I will try to dig again

Demo:

IOS (native) IOS (mWeb)
Simulator.Screen.Recording.-.iPhone.15.-.2024-12-30.at.07.16.29.1.mp4
Simulator.Screen.Recording.-.iPhone.15.-.2024-12-30.at.07.26.30.mp4

@shubham1206agra
Copy link
Contributor

@NJ-2020 What is the current progress on this?

@melvin-bot melvin-bot bot added the Overdue label Jan 12, 2025
@NJ-2020
Copy link
Contributor

NJ-2020 commented Jan 12, 2025

@shubham1206agra Sorry I've been busy with some issues, I will try to fix for Android issue because the loader is displayed while the content is loaded for about 1/1.5 seconds

@melvin-bot melvin-bot bot removed the Overdue label Jan 12, 2025
@NJ-2020
Copy link
Contributor

NJ-2020 commented Jan 13, 2025

Update:

  • I cannot reproduce this issue on android using poor internet:
First attempt Second attempt
test123.mp4
test124.mp4

But inside the second attempt (android native) the empty blocking view inside LHN is shown briefly before the chat list is showed

  • The issue is still reproducible on IOS native
Demo
Simulator.Screen.Recording.-.iPhone.15.-.2025-01-13.at.17.25.45.mp4

@shubham1206agra Could please attempt reproduction on Android native please, and re-confirm?

@NJ-2020
Copy link
Contributor

NJ-2020 commented Jan 23, 2025

@shubham1206agra Bump for the above ^

@shubham1206agra
Copy link
Contributor

@OfstadC Can you add retest label here? And also, can you test hybrid app in this too?

@NJ-2020
Copy link
Contributor

NJ-2020 commented Jan 23, 2025

@shubham1206agra If this issue is not reproducible in Android I think we can still fix it in IOS, web and desktop only

@OfstadC OfstadC added the Needs Reproduction Reproducible steps needed label Jan 27, 2025
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@shubham1206agra
Copy link
Contributor

@OfstadC I think you put the wrong label here

@OfstadC OfstadC added retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause and removed Needs Reproduction Reproducible steps needed labels Jan 27, 2025
@OfstadC
Copy link
Contributor

OfstadC commented Jan 27, 2025

@OfstadC I think you put the wrong label here

Sorry 🤦 - Monday is Monday'ing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2
Projects
Status: MEDIUM
Development

No branches or pull requests

10 participants