Web/Chrome - Broken emoji under Frequently used section

[kbecciv]

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. Go to staging.new.expensifail.com
2. Log in with applausetester+0901abb@applause.expensifail.com/ Feya86Katya
3. Select Chat Kate Becciv and Click in Emoji

Expected Result:

Correct emoji should be displayed

Actual Result:

Broken emoji under Frequently used section

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.84.11

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): applausetester+0901abb@applause.expensifail.com/ Feya87Katya

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

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[melvin-bot]

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

[Beamanator]

@kbecciv I wasn't able to log in with that account :(

[Beamanator]

I searched through my emojis and couldn't figure out which one that is - can you try to insert the broken one into a chat and then copy / paste it here maybe? 🤔

[kbecciv]

@Beamanator Sorry, I was testing something yesterday and changed my password. Please try again this credentials:
applausetester+0901abb@applause.expensifail.com/Feya86Katya

[mvtglobally]

Additional discussion here https://expensify.slack.com/archives/C01GTK53T8Q/p1657873845101639

[luacmartins]

Likely caused by https://github.com/Expensify/Web-Expensify/pull/34210

[Beamanator]

Hmm ok so I wonder if there was some weird bug but now it's fixed? I set my NVP to an empty array and now it seems it's working... I will ask around to see if others fix it with this - if so, I may write a CQ to clear out everyone's NVP (since our App is just in beta still)

[luacmartins]

@Beamanator well this PR was deployed 22h ago, which should have fixed this. In my initial implementation of getUserDetails I used Onyx merge instead of set as a method for frequently used emojis. That caused the array to be duplicated endlessly on each start / reconnect of the app 😅 I believe this was a front-end only issue, so a CQ shouldn't be necessary.

[Beamanator]

Didn't get to this today, will look into it tomorrow or next week

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[luacmartins]

I think we can close this. We didn't manage to repro last week!

[Beamanator]

Yeah I agree, we can reopen if someone produces later 🤷