[HOLD for payment 2024-11-07] [$250] [CVP] OD Navigation - Login - Multiple loading screens and animations are visible after a new user signs up

[isagoico]

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.44-0

Reproducible in staging?: Yes
Reproducible in production?: Yes

If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5043992&group_by=cases:section_id&group_id=327504&group_order=asc

Email or phone of affected tester (no customers): New User
Logs: https://stackoverflow.com/c/expensify/questions/4856

Issue reported by: Applause - Internal team

Action Performed:

1. Navigate to https://staging.expensify.com/
2. Tap on "Organize my own expenses"
3. Input an email address that doesn't have an account yet
4. Tap on the "Get Started" button
5. Tap on the "Join" button

Expected Result:

A single loading animation or a logo should be displayed after the user taps on "Get Started", right until the onboarding shows.

Actual Result:

After new user taps on "Get Started", the "Join" button has to be tapped. White screen is showed, Expensify logo is visible for a short time, small "E" logo with the green background is visible for a short time, "Launching Expensify" is visible for a short time, loading spinner is visible for a short time

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

Bug6623614_1727983756286.EHBM3917.mp4

View all open jobs on GitHub

---

This was found when executing the newly added flows for CVP https://github.com/Expensify/Expensify/issues/412920.

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021843207040930164854
• Upwork Job ID: 1843207040930164854
• Last Price Increase: 2024-10-07

Issue Owner

Current Issue Owner: @Christinadobrzyn

[melvin-bot]

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

[Christinadobrzyn]

I can't get this welcome page to load. I'll test on Monday

[melvin-bot]

Auto-assign attempt failed, all eligible assignees are OOO.

[melvin-bot]

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

[melvin-bot]

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

[Christinadobrzyn]

Okay I can reproduce this. I think this can be external and let's get design on this to get an idea of what we should be showing.

[trjExpensify]

CVP issues go in f1! 👍

[waterim]

Hello, Im Artem from Callstack and would like to help with this issue!

[Christinadobrzyn]

Hi @waterim - sounds great! I added you to the issue

[waterim]

To be clear Im not sure that the expected result is a real expected result

Detailed summary from my perspective regarding the actual results:

After new user taps on "Get Started", the "Join" button has to be tapped. White screen is showed, Expensify logo is visible for a short time, small "E" logo with the green background is visible for a short time, "Launching Expensify" is visible for a short time, loading spinner is visible for a short time

Actual result looks quite okay excepted of the one blink page:

1. the "Join" button has to be tapped - its needed for a verification if user exists or not, if its a new user "Join appears"
2. White screen is just a transition between 2 different pages in 2 different apps (from oldDot to newDot)
3. Expensify logo is a loading state for, but can't reproduce this one, only have a white screen after the join and after E logo
4. E Logo is a correct one as a loader for newDot
5. "Launching Expensify, your session is expired" - this one looks like a bug and from what I can see the response is undefined for the first call (actually in the network there is only one call and it has a response)

And here after we got an undefined we are setting signedInWithShortLivedAuthToken: null

And after we are getting a correct response with all the data, but because we had an signedInWithShortLivedAuthToken as null for a second we have this "session is expired" for a moment.

6. loading spinner is a correct state when we've got a new data for the SignInWithShortLivedAuthToken

It looks like a backend issue as we are getting an undefined for a moment after a call and all other steps are looking fine from a perspective we are making a transition from oldDot to a newDot, and we cant do this with one loader because different websites are rendering

[carlosmiceli]

@waterim Oh, just read your comment, I'll look into in the BE and report back if we need to fix anything here. Thank you! 💪

[carlosmiceli]

Mmm, I'm 99% sure that there's no way to do that, right @trjExpensify? If the issue is in ND but you absolutely need to have access to OD, then maybe I should grab the ND side of this issue as well, but I'll let Tom confirm!

[trjExpensify]

That's a good question for #engineering-chat. 👍

[carlosmiceli]

Yeah, good call! @waterim Just asked and if you can confirm that this can't be done without access to OD, then I'll have to grab it because there's no way to provide access there for the moment.

[waterim]

From my side it looks like impossible to test it properly, I tried to debug just with staging, but can't log anything or test without redirecting to a local newDot, but from my side it looks like we don't want to set signedInWithShortLivedAuthToken: null as this one is causing this "session is expired" and playing with Onyx data for this specific case can be a good solution imo, but cant test to prove this solution unfortunately :(

[carlosmiceli]

Right, that's a bummer! No worries, I'll grab it and will let you know how it goes and if you can help out as well 🙏
Thank you!

[carlosmiceli]

Didn't get to this yet, should be able to work on it tomorrow.

[carlosmiceli]

The fix ended up being a lot simpler, we needed to use account from Onyx, not as a prop: #51450

[Christinadobrzyn]

PR in the works! @carlosmiceli can we move this to weekly while the PR is being worked on?

[carlosmiceli]

Sorry, what needs to be worked on? All PR fixes have been merged for this.

[carlosmiceli]

But yes, we can move this to weekly until it's closed!

[Christinadobrzyn]

Oh perfect! Sorry I meant move to weekly while the PR is going into production. Thanks!

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.55-10 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Remove session expired screen flashing while signing in #51450

If no regressions arise, payment will be issued on 2024-11-07. 🎊

For reference, here are some details about the assignees on this issue:

• @waterim does not require payment (Contractor)
• @eh2077 requires payment through NewDot Manual Requests

[melvin-bot]

@eh2077 @Christinadobrzyn The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]

[eh2077]

C+ wasn't involved in this task, so no payment needed here and unassigning myself

[Christinadobrzyn]

Do we need a regression test for this?

[Christinadobrzyn]

Dmd with @carlosmiceli no regression needed here so closing!