Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.16k stars 2.65k forks source link

[$250] mWeb - Chat - Letter sent with emoji is not displayed #42647

Closed lanitochka17 closed 1 month ago

lanitochka17 commented 1 month ago

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


Version Number: 1.4.76 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4575388 Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. Tap on a report
  3. Tap emoji picker
  4. Select an emoji and enter a letter
  5. Send the message

Expected Result:

Emoji with a letter sent must be displayed

Actual Result:

Emoji with a letter is sent but letter is not displayed, only emoji can be seen

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/99e34ccc-7f8e-4386-8d38-f198bdd8ac4f

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01fe9b72a4d46f8613
  • Upwork Job ID: 1796545237013188608
  • Last Price Increase: 2024-06-14
Issue OwnerCurrent Issue Owner: @allgandalf
melvin-bot[bot] commented 1 month ago

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.

lanitochka17 commented 1 month ago

@sonialiap FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

lanitochka17 commented 1 month ago

We think that this bug might be related to #vip-vsp

lanitochka17 commented 1 month ago

We also observed an issue with a lost letter in another situation:

  1. Go to https://staging.new.expensify.com/
  2. Tap on a report
  3. Enter 123123

https://github.com/Expensify/App/assets/78819774/b43be59f-904a-4971-87f7-7c64332c64ef

Is this the same issue or a new one?

melvin-bot[bot] commented 1 month ago

@sonialiap Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

sonialiap commented 1 month ago

Odd! Adding External

melvin-bot[bot] commented 1 month ago

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

melvin-bot[bot] commented 1 month ago

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

ashuvssut commented 1 month ago

I tested in Edge browser (a Chromium based browser) in my Android device and found that its not reproducible

allgandalf commented 1 month ago

not overdue, waiting for proposals...

allgandalf commented 1 month ago

@lanitochka17 @sonialiap , can you please ask the QA team to retest this bug and see if it is reproducible, last week i was able to reproduce this 2/10 times, but not able to reproduce this week, thanks

lanitochka17 commented 1 month ago

Issue is still reproducible on the Redmi note 10s/Chrome

https://github.com/Expensify/App/assets/78819774/b80f9377-6bd8-4f75-aa51-e5668936bfd8

sonialiap commented 1 month ago

Thanks Svetlana

allgandalf commented 1 month ago

@sonialiap , i updated my GH username, can you please update the issue owner accordingly, new name is : @allgandalf

allgandalf commented 1 month ago

[!NOTE] Issue is consistently reproducible on android web, make sure that you have not focused previously on the compose box

https://github.com/Expensify/App/assets/110545952/f0c983fd-965a-4809-8903-04237c8832a3

Also note that please include in your proposal why this issue is only reproducible on Android web and not any other platfroms, thanks 🙏

sonialiap commented 1 month ago

@allgandalf I think GH automatically made the update

image
allgandalf commented 1 month ago
Screenshot 2024-06-05 at 11 28 03 PM

@sonialiap , i was referring to the issue description, here still my old username is visible :)

sonialiap commented 1 month ago

😄 updated

melvin-bot[bot] commented 1 month ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

mvtglobally commented 1 month ago

Issue not reproducible during KI retests. (First week)

melvin-bot[bot] commented 1 month ago

@sonialiap @allgandalf this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

allgandalf commented 1 month ago

Still waiting for proposals........

allgandalf commented 1 month ago

Same as ^

melvin-bot[bot] commented 1 month ago

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

allgandalf commented 1 month ago

Waiting for proposals .....

sonialiap commented 1 month ago

No longer reproducible

https://github.com/Expensify/App/assets/17131195/690612ac-daab-41ee-aa09-e8b4abab3b50

allgandalf commented 1 month ago

@sonialiap , this issue is reproducible on Android web and not desktop, can you please reopen it

sonialiap commented 1 month ago

@allgandalf I can't reproduce it on Android web Chrome either. The letter comes through after the emoji both when there's a space after the emoji or when I delete the space and add the letter directly after

https://github.com/Expensify/App/assets/17131195/a2ac25fb-7871-4eae-bd2e-70b33b605433