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

[Due for payment 2025-02-27] [Due for payment 2025-02-27] Android - Reports - App crashes when switching workspaces on reports section. #57076

Closed
1 of 8 tasks
mitarachim opened this issue Feb 19, 2025 · 16 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2

Comments

@mitarachim
Copy link

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.1.1-0
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: https://expensify.testrail.io/index.php?/tests/view/5618498&group_by=cases:section_id&group_order=asc&group_id=296775
Email or phone of affected tester (no customers): ibellicotest+264@gmail.com
Issue reported by: Applause Internal Team
Device used: Motorola MotoG60 - Android 12
App Component: Search

Action Performed:

  1. Open the Expensify app.
  2. Tap on "Reports" on the bottom of the screen.
  3. Scroll the filters to the right, and open any of them. (For example: Done)
  4. Tap on the workspace switcher.
  5. Switch to any workspace.
  6. Tap on workspace switcher again, and change to any other workspace.
  7. If app didn´t crash, repeat steps 4 to 6 a few more times.

Expected Result:

The user should be able to switch to another workspace on reports section without any issue.

Actual Result:

App crashes when switching workspaces on reports section, after selecting a filter.

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6747506_1739946485015!log.txt
https://github.com/user-attachments/assets/1a9fa5d5-8c78-4cfa-a123-a4113ffa4a46

View all open jobs on GitHub

@mitarachim mitarachim added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 DeployBlocker Indicates it should block deploying the API DeployBlockerCash This issue or pull request should block deployment labels Feb 19, 2025
Copy link

melvin-bot bot commented Feb 19, 2025

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

Copy link

melvin-bot bot commented Feb 19, 2025

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

Copy link

melvin-bot bot commented Feb 19, 2025

💬 A slack conversation has been started in #expensify-open-source

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.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Feb 19, 2025
@mitarachim
Copy link
Author

mitarachim commented Feb 19, 2025

we can repro issue after repeat steps 4 to 6 in 7 times or more

@mjasikowski
Copy link
Contributor

I'll try to reproduce

@jnowakow
Copy link
Contributor

I was asked by @SzymczakJ to reproduce this one and I was able to do so.

App crashes with this error from Reanimated: ``` 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A Cmdline: org.me.mobiexpensifyg.dev 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A pid: 15510, tid: 15739, name: mqt_v_js >>> org.me.mobiexpensifyg.dev <<< 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #00 pc 00000000001775cc /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (worklets::ReanimatedReentrancyCheck::before()+324) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #1 pc 000000000017745c /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (facebook::jsi::detail::BeforeCaller::before(worklets::ReanimatedReentrancyCheck&)+36) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #2 pc 00000000001773b4 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (facebook::jsi::WithRuntimeDecorator::Around::Around(worklets::ReanimatedReentrancyCheck&)+52) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #3 pc 00000000001693bc /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (facebook::jsi::WithRuntimeDecorator::createObject()+52) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #4 pc 00000000001e9588 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libreanimated.so (facebook::jsi::Object::Object(facebook::jsi::Runtime&)+60) (BuildId: 979f3a42cfa24c9b3571892ef0eb7590109cd447) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #5 pc 00000000000eaa30 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (worklets::ShareableObject::toJSValue(facebook::jsi::Runtime&)+60) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #6 pc 00000000000eb5e0 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (worklets::ShareableWorklet::toJSValue(facebook::jsi::Runtime&)+180) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #7 pc 0000000000281a44 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libreanimated.so (facebook::jsi::Value worklets::WorkletRuntime::runGuarded<>(std::__ndk1::shared_ptr const&) const+92) (BuildId: 979f3a42cfa24c9b3571892ef0eb7590109cd447) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #8 pc 000000000018aa2c /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (worklets::WorkletRuntime::runAsyncGuarded(std::__ndk1::shared_ptr const&)::'lambda'()::operator()() const+60) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #9 pc 000000000018a9c4 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #10 pc 000000000018a91c /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (void std::__ndk1::__invoke_void_return_wrapper::__call const&)::'lambda'()&>(worklets::WorkletRuntime::runAsyncGuarded(std::__ndk1::shared_ptr const&)::'lambda'()&)+36) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #11 pc 000000000018a8cc /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #12 pc 0000000000188e68 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (std::__ndk1::__function::__func const&)::'lambda'(), std::__ndk1::allocator const&)::'lambda'()>, void ()>::operator()()+40) (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #13 pc 000000000030eb7c /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libreanimated.so (BuildId: 979f3a42cfa24c9b3571892ef0eb7590109cd447) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #14 pc 000000000030eaf0 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libreanimated.so (std::__ndk1::function::operator()() const+36) (BuildId: 979f3a42cfa24c9b3571892ef0eb7590109cd447) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #15 pc 0000000000112530 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #16 pc 000000000011233c /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #17 pc 0000000000112240 /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.958 15763-15763 DEBUG pid-15763 A #18 pc 00000000001117ac /data/app/~~FdiOdTZ8txEMwxtB409GVw==/org.me.mobiexpensifyg.dev-yM7ImqzX9AvhW2QbkFZGXQ==/lib/arm64/libworklets.so (BuildId: 7fab2e0fdec9300d6010b0bb2f675cf66f00dcde) 2025-02-19 12:11:06.979 541-15770 ActivityTaskManager system_server W Force finishing activity org.me.mobiexpensifyg.dev/org.me.mobiexpensifyg.ExpensifyActivityBase 2025-02-19 12:11:06.992 541-558 ActivityManager system_server I Showing crash dialog for package org.me.mobiexpensifyg.dev u0 ```

After quick conversation with @tomekzaw it occurred that he's aware of this problem it was already fixed in the library. We will apply those changes as patch as we are using older version of reanimated

@tomekzaw
Copy link
Contributor

Here's the cherry-pick for Reanimated 3.17.0: software-mansion/react-native-reanimated@fea63f0

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Hourly KSv2 labels Feb 19, 2025
@luacmartins luacmartins removed the DeployBlocker Indicates it should block deploying the API label Feb 19, 2025
Copy link

melvin-bot bot commented Feb 19, 2025

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@luacmartins
Copy link
Contributor

This is fixed in staging. Closing.

@luacmartins luacmartins removed the DeployBlockerCash This issue or pull request should block deployment label Feb 19, 2025
@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Feb 20, 2025
@melvin-bot melvin-bot bot added the Awaiting Payment Auto-added when associated PR is deployed to production label Feb 20, 2025
@melvin-bot melvin-bot bot changed the title Android - Reports - App crashes when switching workspaces on reports section. [Due for payment 2025-02-27] Android - Reports - App crashes when switching workspaces on reports section. Feb 20, 2025
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Feb 20, 2025
Copy link

melvin-bot bot commented Feb 20, 2025

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

Copy link

melvin-bot bot commented Feb 20, 2025

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

If no regressions arise, payment will be issued on 2025-02-27. 🎊

Copy link

melvin-bot bot commented Feb 20, 2025

@mjasikowski @mjasikowski 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]

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Feb 20, 2025
@melvin-bot melvin-bot bot changed the title [Due for payment 2025-02-27] Android - Reports - App crashes when switching workspaces on reports section. [Due for payment 2025-02-27] [Due for payment 2025-02-27] Android - Reports - App crashes when switching workspaces on reports section. Feb 20, 2025
Copy link

melvin-bot bot commented Feb 20, 2025

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

If no regressions arise, payment will be issued on 2025-02-27. 🎊

Copy link

melvin-bot bot commented Feb 20, 2025

@mjasikowski @mjasikowski 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]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Engineering Weekly KSv2
Projects
None yet
Development

No branches or pull requests

7 participants