[HOLD for payment 2024-11-18] [HOLD for payment 2024-11-14] Android -Copilot-App crashes when adding a copilot

[IuliiaHerets]

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: v9.0.58-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Issue was found when executing this PR: #51588
Issue reported by: Applause Internal Team

Action Performed:

1. Go to settings > security
2. Tap Add copilot
3. Enter the email address of any user

Expected Result:

The loading icon is displayed for a few seconds and the user is found.
After selecting the user found, the flow can continue.

Actual Result:

Infinite loading when entering the email address and a few seconds later the system message "Expensify isn't responding" is displayed.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6656163_1730869560498.Adding_a_copilot.mp4

View all open jobs on GitHub

Issue Owner

Current Issue Owner: @mjasikowski

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[github-actions]

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

[IuliiaHerets]

@mjasikowski @RachCHopkins App also crashed when we invited a new member to a workspace.
Do we need to raise a new issue or it is the same root as this?

Bug6656281_1730880406269.video_2024-11-06_11-02-58.mp4

[mjasikowski]

@IuliiaHerets the root cause is most likely the same and is currently being investigated, but please raise a new issue so we won't miss the new problem when testing for a fix

[mjasikowski]

This is investigated and discussed in Slack

[kavimuru]

Fixed

Uploading 51588-No Repro.mp4…

[melvin-bot]

⚠️ 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.

[melvin-bot]

⚠️ 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.

[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.58-2 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [CP Staging] Revert "Feature: add loading indicator when ReconnectApp is running" #52112

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

[melvin-bot]

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

[RachCHopkins]

@mjasikowski who do I need to pay here?

[mjasikowski]

@RachCHopkins This was fixed internally with the help of Applause as far as I see, so no one needs to be paid.

[RachCHopkins]

Ok cool, in that case do we need to do the checklist @mjasikowski ?

[melvin-bot]

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

• [CP Staging] Revert "Feature: add loading indicator when ReconnectApp is running" #52112

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

[melvin-bot]

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

[mjasikowski]

@RachCHopkins This was a one-off related to other crashes, so we can skip the checklist here. I'll close this one.