-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
iOS Hybrid - App crashes when switching to Expensify Classic after logging in to a new account #54169
Comments
Triggered auto assignment to @michaelkwardrop ( |
Triggered auto assignment to @luacmartins ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 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:
|
@luacmartins , is this being addressed in the fireroom? |
This issue is being discussed here |
This SO doesn't mention anything about adding these to a project, is that something that is not needed in this case? Or is it best practice to always have a |
@IuliiaHerets I'm unable to reproduce the crash. Could you please ask for a re-test? |
I'm currently stuck trying to run hybrid app locally. Discussing it here. Additionally:
|
I'm also unable to reproduce on my phone on testflight |
@luacmartins Tester still can repro the issue, build - 76-6 ScreenRecording_12-17-2024.04-27-01_1.MP4 |
I can reproduce this crash on my device too, iPhone XR/iOS 17.3 IMG_8598.MP4 |
We're still discussing this issue here, but we can't rule out the API side of things yet either since we can't really reproduce it internally and don't really know what's causing the issue. |
I have to head out now, but I'll be back in ~4h to continue investigating |
What's the latest on this? |
No meaningful progress yet. We haven't been able to reproduce the issue internally. Additionally:
|
I also tried to reproduce the issue, but I was unable to do that. Maybe it's an issue with a specific account type? |
I can reproduce this consistently. |
Ughh, with a new test user it only crashes on first switch... frustrating because previously it reproduced on every switch. @grgia when I reopened the app after crash I land in OldApp experience, then had to switch twice to reproduce crash. |
I don't see any step that requires mWeb for testing this flow, am I missing it?
I authenticated on web, then again on HybridApp |
no you're right I think I misinterpreted this one @Julesssss |
The sign-out case sounds slightly different, but yeah that's the post-crash Modal I was seeing. |
@grgia that's Android crashes, you need to switch to iOS HybridApp release, but yeah it's useless. This version code isn't even available in the filter in Crashlitics, as it detected zero crashes. Also it's the previous version -- I think you're looking at the big Android crash from the weekend. There's the JS crash dashboard Andrew and Carlos were talking about, but it's also not very helpful. |
I see a couple of crashes in the iOS Firebase console after removing the top level Still though, nothing is reported against version |
Have we confirmed if this is only impacting |
It looks like just |
@Julesssss is there a way to search the Apple Developer crash (from the popup on IOS) via searching my email or comment? Where do those go? |
@luacmartins my version is |
Yea, there's nothing for 9.0.76.X. The last report I see is from Nov 29 😂 |
We have a fireroom for this one https://expensify.enterprise.slack.com/archives/C085GD1ET8A |
Going to remove the web deploy blocker label per discussion here |
@mvtglobally was this crash found during regular regression tests? |
@luacmartins this is not linked to a particular TC steps but rather found while running regression as an exploratory issue. |
@mvtglobally can you test the latest build |
I cannot reproduce on the latest build, yesterday in my device it was repro (iPhone XR/iOS 17.3) IMG_8601.MP4 |
Thanks for confirming! This issue is fixed. Closing. |
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.76-6
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): applausetester+23082038203a2@applause.expensifail.com
Issue reported by: Applause Internal Team
Device used: iPhone 15 Pro Max / iOS 18.2
App Component: Other
Action Performed:
Precondition:
Expected Result:
App will not crash.
Actual Result:
App crashes when switching to Expensify Classic after logging in to a new account.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6694850_1734270273609!Expensify-2024-12-15-212502.txt
Bug6694850_1734270273607.20241215_212417.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: