[HybridApp] Users stuck at white screen during app open

[Julesssss]

Hybrid app is unresponsive for HybridApp users set to the NewDot experience.

Internal users are locked or held on a white screen when opening the app or signing in. This seems to occur for users set to the NewDot experience. Two methods of reproduction are listed below. Method A seems to be occasionally reproducable for users immediately after switching experiences, wheras method B seems to be easy to reproduce for large accounts or on bad connections.

There are two API commands that fail: https://www.expensify.com/_devportal/tools/logSearch/#sort=asc&size=10000&query=%22Error%20processi[…]024-07-25T00%3A00%20TO%202024-08-02T23%3A59%5D

I think this causes us to force a logout and see the white screen. it seems like a race condition of some sorts, and I do agree that it seems to be happening because of a redirect issue. I think we should make an issue and dive deeper into it while doing the rollout

---

Version Number: v9.0.13.4
Reproducible in staging?: Y
Reproducible in production?: Y
Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1722805554897749?thread_ts=1722276993.634759&cid=C036QM0SLJK

Action Performed:

Method A

• Install the Hybrid app, version 9.0.13.4
• Open app not logged in
• Authenticate with an Expensify account via magic code
• Redirected to NewDot experience
• Switched back to Expensify classic from NewDot settings
• Signed out of OldDot experience
• Tried to create new account

Method B -- not 100% reproducable

• Have the app set to NewDot experience
• Open the app
• It often hangs
• Sometimes after a long time you land in the app

Workaround:

Users without redirection (non-internal users) shouldn't be affected. Switch to OldDot experience to resolve

None

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

Method A:

https://drive.google.com/file/d/141Ob-O4f1iGaikI6cAJPOQ8nx1_z92Th/view

Method B:

screen-20240805-075239.mp4

[Julesssss]

@trjExpensify, please prioritise accordingly. Prior to release we didn't hold on this issue, and it should still be isolated to users set to the NewDot experience, but now that it is occurring on app open we may need to rethink.

[trjExpensify]

Yes, we should get to the bottom of this one before we flip the switch for new sign-ups. Who's taking the investigation on the HybridApp team?

[staszekscp]

Hey! I'll investigate what's going on here!

[staszekscp]

Ok, I have an update number 1 😄

It seems that the main problem is the API.GetShortLivedAuthToken() command failing while being offline. Because of that we are not able to open NewDot. We just catch an exception, and display an error snackbar. Instead, I think we should call yaplOpenNewApp there with a different url. This way, we could indicate that we enter the app after an exception. However, we will have to cover well all problematic edge cases (like network problems after switching accounts).

Right now I consider creating a new native method, that would return only email of the current user logged in OldDot. This way we could compare the emails, and if they match, we would let the user into NewDot. Otherwise, we could eg. open OldDot sign in page, or display some Error page in NewDot to "try again later".

[staszekscp]

I'm back with update number 2 😄

I've managed to fix iOS, but I still need to do Android and some testing in order to find loopholes. However, so far it doesn't look bad:

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2024-08-07.at.14.42.07.mp4

[AndrewGable]

I think I heard @war-in and @mateuuszzzzz are working on this one, can you both comment here so I can assign you both? Thanks!

[mateuuszzzzz]

👍

[war-in]

👋

[melvin-bot]

Triggered auto assignment to @francoisl, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[mallenexpensify]

Is @ntdiary due compensation for their review on the below PR? (if so, it's $125 cuz there was no testing)

• [HybridApp] Share auth token in HybridApp #48007

[Julesssss]

Hi @mallenexpensify, yes that's correct. Please help us to pay @ntdiary for the code review and close out this issue.

[mallenexpensify]

Contributor+: @ntdiary due $125 via NewDot

I'm assuming a regression test isn't needed here cuz it's created already as part of HybridApp, comment/reopen if anyone disagrees.

[JmillsExpensify]

$125 approved for @ntdiary