[$500] Hybrid - Android - Chat – App reloads/crashes when select an emoji from suggestions list

[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.82-4
Reproducible in staging?: Yes
Reproducible in production?: No
Issue reported by: Applause Internal Team
Device used: RealmeC67 / Android 14

Action Performed:

1. Go to Hybrid app
2. Log in with Gmail account
3. Open any chat
4. Enter :sm and select any emoji from the suggestions list

Expected Result:

App doesn’t reloads/crashes. Emoji is present in composer

Actual Result:

App reloads/crashes when select an emoji from suggestions list

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

video_2025-01-09_19-21-44.mp4

View all open jobs on GitHub
logs-2025-01-09 17_03_30.307.txt

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021877460008842832724
• Upwork Job ID: 1877460008842832724
• Last Price Increase: 2025-01-09

Issue Owner

Current Issue Owner: @allroundexperts

[melvin-bot]

Triggered auto assignment to @kadiealexander (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 @rlinoz (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]

Production

1.mp4

[rlinoz]

I can repro, so looking

[mountiny]

@WoLewicki is also looking for a bit.

[rlinoz]

Thanks, I am having a hard time building hybrid

[WoLewicki]

I was also trying to build Android in Hybrid for like an hour now and failed. Not sure if something weird was merged, but it fails with react-native-live-markdown right now. Do you see the same problem @rlinoz ?

[rlinoz]

I removed Mobile-Expensify folder and started the whole process again and now it worked 😄

Edit: Actually, the build worked, but I am getting no internet connection in the app

[rlinoz]

Here is the error, nothing is logged on metro though

[rlinoz]

OK the stacktrace is pretty much that

01-09 17:44:19.382  5967  7009 E unknown:ReactHost|ReactNoCrashBridgeNotAllowedSoftException:getNativeModule(UIManagerModule.cl	at org.me.mobiexpensifyg.RNTextInputResetModule.resetKeyboardInput(RNTextInputResetModule.java:34)
01-09 17:44:19.485  5967  7009 E unknown:ReactNative: 	at org.me.mobiexpensifyg.RNTextInputResetModule.resetKeyboardInput(RNTextInputResetModule.java:35)
01-09 17:44:19.488  5967  6986 E unknown:ReactHost: 	at org.me.mobiexpensifyg.RNTextInputResetModule.resetKeyboardInput(RNTextInputResetModule.java:35)

[rlinoz]

I am guessing this is related to the RN update, checking that now, if confirmed I will open up for proposals so we don't have to revert it

[rlinoz]

So I am wrong, it is not reproducible in the adhoc build from that PR

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021877460008842832724

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane (External)

[melvin-bot]

Upwork job price has been updated to $500

[allroundexperts]

https://github.com/allroundexperts/Expensify/blob/3b665f7ff38fc7b3b7155e84eb0d09e680742a82/src/pages/home/report/ReportActionCompose/SuggestionEmoji.tsx#L96 is causing the crash. It's highly probable that this was related to RN upgrade.

[allroundexperts]

So I am wrong, it is not reproducible in the adhoc build from that PR

@rlinoz Which build did you use to test?

[allroundexperts]

Okay. Found the issue. We are missing this commit on hybrid app.

[allroundexperts]

@rlinoz Can I raise a PR or should I post a proper proposal?

[allroundexperts]

And @mountiny a gentle bump on #54988 (comment) 😄

[allroundexperts]

Result:

Screen.Recording.2025-01-10.at.3.38.40.AM.mov

[rayane-djouah]

Great catch @allroundexperts 🚀

[allroundexperts]

PR ready: https://github.com/Expensify/Mobile-Expensify/pull/13353