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] [Waiting for payment - 2024-10-06] iOS - Login - White screen persists for some time after login #49880

Closed
1 of 6 tasks
lanitochka17 opened this issue Sep 28, 2024 · 37 comments · Fixed by #49922
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Reviewing Has a PR in review

Comments

@lanitochka17
Copy link

lanitochka17 commented Sep 28, 2024

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.41-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:

  1. Launch New Expensify app
  2. Log in with an account with many reports

Expected Result:

There will be no white screen after login

Actual Result:

White screen persists for some time after login
On production, there is white screen but it appears very briefly

Workaround:

Unknown

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

Bug6617803_1727467446758.ScreenRecording_09-28-2024_04-00-21_1.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021840741226805286590
  • Upwork Job ID: 1840741226805286590
  • Last Price Increase: 2024-09-30
Issue OwnerCurrent Issue Owner: @
@lanitochka17 lanitochka17 added DeployBlockerCash This issue or pull request should block deployment DeployBlocker Indicates it should block deploying the API labels Sep 28, 2024
Copy link
Contributor

👋 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:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Sep 28, 2024

Triggered auto assignment to @Julesssss (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@roryabraham
Copy link
Contributor

Going to venture a guess that this is a front-end blocker, not a back-end blocker

@roryabraham roryabraham removed the DeployBlocker Indicates it should block deploying the API label Sep 29, 2024
@Julesssss
Copy link
Contributor

This doesn't occur every time on my account, but I can reproduce on 9.0.41.1 and 9.0.41.0.Not 9.0.40.6. however

There are a lot of issues in the checklist. I'm working through them but in the meantime have asked for thoughts in Slack.

@truph01
Copy link
Contributor

truph01 commented Sep 30, 2024

Edited by proposal-police: This proposal was edited at 2024-09-30 11:07:48 UTC.

Proposal

Please re-state the problem that we are trying to solve in this issue.

White screen persists for some time after login

What is the root cause of that problem?

  • Come from PR.
  • We return null when the data is loading:

if (isLoadingOnyxValue(sessionStatus, lastOpenedPublicRoomIDStatus, initialLastUpdateIDAppliedToClientStatus)) {
return;

What changes do you think we should make in order to solve the problem?

  • We should return <FullScreenLoadingIndicator /> instead. (Or another loading screen based on design)

What alternative solutions did you explore? (Optional)

@truph01
Copy link
Contributor

truph01 commented Sep 30, 2024

Proposal updated

@Julesssss
Copy link
Contributor

Thanks @truph01. As this is a blocker I'm testing your proposal here.

@Nodebrute
Copy link
Contributor

@Julesssss Reverting #49185 also fixes this issue

@Julesssss
Copy link
Contributor

#49185

Yeah, in this case a simple fix might be preferable.

@allgandalf
Copy link
Contributor

allgandalf commented Sep 30, 2024

@truph01 's solution seems good actually, @truph01 have you tested it though ?

Seems to be working

@Julesssss
Copy link
Contributor

Yeah, I'm fixing it here.

@truph01 we'll payout 50% for locating the regression.

@Julesssss
Copy link
Contributor

Let's revert to speed things up. We can settle on the fix when re-submitting the PR.

@allgandalf will review

@truph01
Copy link
Contributor

truph01 commented Sep 30, 2024

@truph01 we'll payout 50%

Thanks @Julesssss

@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

Current assignee @allgandalf is eligible for the External assigner, not assigning anyone new.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Sep 30, 2024
@Julesssss Julesssss reopened this Sep 30, 2024
@Nomanahmeda789
Copy link

The full screen loader have some issue.

This comment was marked as resolved.

@Nomanahmeda789

This comment was marked as outdated.

This comment was marked as outdated.

@blazejkustra
Copy link
Contributor

The revert doesn't fully fix the bug, when the internet speed is really bad you can still reproduce the bug, for a brief second white screen is shown after logging in:

Screen.Recording.2024-09-30.at.15.39.59.mov

@Nomanahmeda789

This comment was marked as resolved.

This comment was marked as resolved.

@Julesssss
Copy link
Contributor

The revert doesn't fully fix the bug, when the internet speed is really bad you can still reproduce the bug

Thanks for sharing. It's far more noticeable, though, after these changes. With a medium-large sized the issue is not seen once reverting the changes.

@Julesssss Julesssss removed the DeployBlockerCash This issue or pull request should block deployment label Sep 30, 2024
@Julesssss
Copy link
Contributor

Julesssss commented Sep 30, 2024

PR that made this blocker noticable has been reverted, deploy blocker label removed.

@blazejkustra
Copy link
Contributor

@Julesssss Reverting it was the right call, but I think we still need to investigate why the white screen is displayed 🤔

@Julesssss
Copy link
Contributor

Yes of course. I'll create a new issue for that, as it shouldn't be treated as a prior blocker.

@Julesssss
Copy link
Contributor

Created this issue to figure out the cause of the blank page issue.

Copy link

melvin-bot bot commented Oct 7, 2024

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

@allgandalf
Copy link
Contributor

Lets close this !?

@truph01
Copy link
Contributor

truph01 commented Oct 8, 2024

@Julesssss Sorry, do I and @allgandalf have payment based on comment?

@Julesssss Julesssss added the Bug Something is broken. Auto assigns a BugZero manager. label Oct 8, 2024
Copy link

melvin-bot bot commented Oct 8, 2024

Triggered auto assignment to @JmillsExpensify (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

@allgandalf were you paid yet? I think probably not right?

@JmillsExpensify could you please pay out the following once this has been confirmed? Thanks

@johncschuster
Copy link
Contributor

Looks like @allgandalf needs to be paid out via Upwork for the review. I'm taking care of that in this issue.

@allgandalf
Copy link
Contributor

Yeah, that is why i asked to close this issue 😄

@Julesssss
Copy link
Contributor

Oh right.

@johncschuster would you mind taking care of the other payment there too?

@truph01 $125 for locating blocker (50%)

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. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors Reviewing Has a PR in review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants