Android - Copilot - User logged as copilot is able to switch to Expensify Classic.

[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: 9.0.73-0 Reproducible in staging?: Yes Reproducible in production?: Yes If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both Issue was found when executing this PR: https://github.com/Expensify/App/pull/52103 Email or phone of affected tester (no customers): [email protected] Issue reported by: Applause Internal Team

Action Performed:

1. Open the Expensify app.
2. Log in with an account that is a Copilot.
3. Tap on "Settings"
4. Scroll down and tap on "Switch to Expensify Classic"
5. Verify that a "Not so fast" message is displayed and that you are not able to switch to Expensify Classic.

Expected Result:

User logged as a copilot, shouldn´t be able to switch to Expensify Classic.

Actual Result:

User logged as a copilot is able to switch to Expensify Classic and the "Not so fast" message is not displayed.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

https://github.com/user-attachments/assets/47e7f9aa-b211-4eba-9072-26d88d53ad49

View all open jobs on GitHub

[melvin-bot[bot]]

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

[ChavdaSachin]

@IuliiaHerets I think this issue should be present on android/ios hybrid app and not on any of the standalone app.

[ChavdaSachin]

@sonialiap it's coming from my pr, Issue should be limited to the Hybrid apps. And adding following code block here should solve the issue

                                  if (isActingAsDelegate) {
                                      setIsNoDelegateAccessMenuVisible(true);
                                      return;
                                  }

Since I am external contributor, I don't have access to hybrid app and I could not test it, and for the same reason the case was not handled in my pr.

[c3024]

I wonder why it is failing on standalone app. This should not fail on standalone app, unless standalone app too returns true for NativeModules.HybridAppModule.

[ChavdaSachin]

I checked android-standalone, but it is working as expected. My guess is that tester might have marked standalone by mistake.

[IuliiaHerets]

@ChavdaSachin I asked the tester to recheck. I'll be back soon with an answer

[IuliiaHerets]

@ChavdaSachin Yes, you are right. Issue happens only on the hybrid app. Sorry for confusing

[sonialiap]

posted in slack for an internal engineer

[Julesssss]

It should be a simple HybridApp change, but sorry, I can't commit to this. Ignore my brief assignment

[sonialiap]

In a slack discussion it was agreed that we're ultimately moving towards allowing Copilot to take all actions so this is maybe not yet expected, but not a bug that we need to fix. Since it's not necessary to fix, closing

[dangrous]

As discussed in the issue, we think this is probably fine - once https://github.com/Expensify/Mobile-Expensify/pull/13289 is merged we should be able to switch back and forth between old and new dot as a copilot. The only reason to try to fix this, I think, is if we think it's a dealbreaker to be able to switch to old dot and not back, for a limited time. We won't be able to switch back until the PR I linked is merged.

Thoughts?

If so, we can make the change @ChavdaSachin mentioned here, happy to act as the engineer on this issue

[Julesssss]

so this is maybe not yet expected, but not a bug that we need to fix. Since it's not necessary to fix, closing

Agree this isn't necessary.