Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chat - Broken multiple Emoji icons #20856

Closed
2 of 6 tasks
kbecciv opened this issue Jun 15, 2023 · 28 comments
Closed
2 of 6 tasks

Chat - Broken multiple Emoji icons #20856

kbecciv opened this issue Jun 15, 2023 · 28 comments
Assignees
Labels

Comments

@kbecciv
Copy link

kbecciv commented Jun 15, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Hover over a text or attachment on the chat
  2. move to Add reaction button
  3. Click on the Add Reaction button
  4. Look at Smiley_face_with_tears emoji icon/image, also disguised_face and more than 20 others

Expected Result:

Smiley_face_with_tears icon/ image to be seen.

Actual Result:

it is just broken icon/image.

Workaround:

Unknown

Platforms:

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

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.3.25.7

Reproducible in staging?: n/a

Reproducible in production?: n/a

If this was caught during regression testing, add the test name, ID and link from TestRail:

Email or phone of affected tester (no customers):

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

Notes/Photos/Videos: Any additional supporting documentation

New.Expensify.-.Google.Chrome.2023-06-08.15-56-19.1.mp4

Expensify/Expensify Issue URL:

Issue reported by: @Amnex-08

Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686229620862939

View all open jobs on GitHub

@kbecciv kbecciv added Daily KSv2 Needs Reproduction Reproducible steps needed Bug Something is broken. Auto assigns a BugZero manager. labels Jun 15, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jun 15, 2023

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot
Copy link

melvin-bot bot commented Jun 15, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@kbecciv kbecciv changed the title Broken multiple Emoji icons Chat - Broken multiple Emoji icons Jun 15, 2023
@melvin-bot melvin-bot bot added the Overdue label Jun 19, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jun 19, 2023

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

@mallenexpensify
Copy link
Contributor

Unable to reproduce, @kbecciv can you test please? Updated OP to inc. reporter - @Amnex-08

@melvin-bot melvin-bot bot removed the Overdue label Jun 19, 2023
@jeet-dhandha
Copy link
Contributor

Looks like Window's issue. Unicode for emojis might not be available for windows

@melvin-bot melvin-bot bot added the Overdue label Jun 22, 2023
@mallenexpensify
Copy link
Contributor

Tested on Windows, Edge 13(?) via Browserstack and wasn't able to repro.
image

image

@Amnex-08, what OS are you on?

@melvin-bot melvin-bot bot removed the Overdue label Jun 22, 2023
@Amnex-08
Copy link

Amnex-08 commented Jun 23, 2023 via email

@melvin-bot
Copy link

melvin-bot bot commented Jun 23, 2023

📣 @Amnex-08! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

  1. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  2. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  3. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
    Screen Shot 2022-11-16 at 4 42 54 PM
    Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

@Amnex-08
Copy link

Contributor details
Your Expensify account email: [email protected]
Upwork Profile Link: (https://www.upwork.com/freelancers/~01d187c1cb0be7fb95)

@melvin-bot
Copy link

melvin-bot bot commented Jun 23, 2023

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@mallenexpensify
Copy link
Contributor

@kbecciv , can you test to see if you can reproduce?

I am on Windows 10 version 22H2 and using Google Chrome version 9 Version
114.0.5735.134 (Official Build) (64-bit).*

@melvin-bot melvin-bot bot added the Overdue label Jun 26, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jun 27, 2023

@mallenexpensify, @kbecciv Whoops! This issue is 2 days overdue. Let's get this updated quick!

@mallenexpensify
Copy link
Contributor

@kbecciv , can you please test again?

@melvin-bot melvin-bot bot removed the Overdue label Jun 28, 2023
@kbecciv
Copy link
Author

kbecciv commented Jun 28, 2023

@mallenexpensify I will do it tomorrow, sorry for delay.

@kbecciv
Copy link
Author

kbecciv commented Jun 28, 2023

@mallenexpensify Unable to reproduce it with build 1.3.33.4

Recording.3289.mp4

@melvin-bot
Copy link

melvin-bot bot commented Jun 29, 2023

@mallenexpensify @kbecciv this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@mallenexpensify
Copy link
Contributor

mallenexpensify commented Jun 30, 2023

K, thanks for testing @kbecciv , gonna close then, comment/reopen if disagree

@mallenexpensify mallenexpensify added Daily KSv2 and removed Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 26, 2023
@mallenexpensify mallenexpensify added Task and removed Daily KSv2 Needs Reproduction Reproducible steps needed labels Sep 26, 2023
@melvin-bot melvin-bot bot added the Overdue label Oct 4, 2023
@mallenexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot removed the Overdue label Oct 4, 2023
@melvin-bot melvin-bot bot added the Overdue label Oct 13, 2023
@mallenexpensify
Copy link
Contributor

on hold

@melvin-bot melvin-bot bot removed the Overdue label Oct 14, 2023
@melvin-bot melvin-bot bot added the Overdue label Oct 23, 2023
@mallenexpensify
Copy link
Contributor

PR for issue this is held on is up for review

@melvin-bot melvin-bot bot removed the Overdue label Oct 25, 2023
@melvin-bot melvin-bot bot added the Overdue label Nov 3, 2023
@mallenexpensify
Copy link
Contributor

Above PR is moving along

@melvin-bot melvin-bot bot removed the Overdue label Nov 3, 2023
@melvin-bot melvin-bot bot added the Overdue label Nov 13, 2023
@mallenexpensify mallenexpensify changed the title [HOLD #21923] Chat - Broken multiple Emoji icons Chat - Broken multiple Emoji icons Nov 14, 2023
@mallenexpensify mallenexpensify added the Daily KSv2 label Nov 14, 2023
@melvin-bot melvin-bot bot removed the Overdue label Nov 14, 2023
@mallenexpensify mallenexpensify removed the Weekly KSv2 label Nov 14, 2023
@mallenexpensify
Copy link
Contributor

Off hold, the PR hit production 4 days ago.

@lanitochka17 can you see if you're able to reproduce the bug on Windows? I don't have a PC available (and using BrowserStack takes a bit, if/when possible)

@lanitochka17
Copy link

@mallenexpensify Hello
Issue is not reproducible on the latest build 1.4.0-1
Windows10/Chrome

Recording.477.mp4

@mallenexpensify
Copy link
Contributor

Thanks @lanitochka17 , going to close then since it's not reproducible

Copy link

melvin-bot bot commented Nov 17, 2023

@mallenexpensify Be sure to fill out the Contact List!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants