Android - Setting - App crashes on tapping view keyboard shortcuts

[kavimuru]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Launch the app
2. Log in with any account
3. Go to Setting > About
4. Click " View Keyboard Shortcuts"

Expected Result:

Information about keyboard shortcut should appear

Actual Result:

App crashes all the time

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Android

Version Number: v1.1.23-0

Reproducible in staging?: Yes

Reproducible in production?: option not available

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:

Bug5384460_20211222_193817.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:

View all open jobs on GitHub

[OSBotify]

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

[MelvinBot]

Triggered auto assignment to @tylerkaraszewski (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[parasharrajat]

Cc: @akshayasalvi this should not be visible on native devices.

[akshayasalvi]

Noted. @parasharrajat I've raised a PR to fix this.

[iwiznia]

Cc: @akshayasalvi this should not be visible on native devices.

Why? This should work on native devices too, no? Why restrict it? It should just work.

[parasharrajat]

Yeah, Now we can rethink this after #6883. I said so as we didn't have keyboard shortcuts for native.

[parasharrajat]

But definitely crash should not happen as It is just a modal and should work on native devices too. Meantime, I have requested the hold on the PR.

[roryabraham]

Just merged #6881 to unblock deploys, but I agree we should enable keyboard shortcuts on native in #6883. I thought it would be better to unblock deploys in the meantime so @jasperhuangg can take his time with #6883