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

[HOLD #14676] Chat - Strikethrough markdown is applied to emojis on compose box #43322

Open
2 of 6 tasks
kavimuru opened this issue Jun 7, 2024 · 14 comments
Open
2 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Monthly KSv2 Needs Reproduction Reproducible steps needed

Comments

@kavimuru
Copy link

kavimuru commented Jun 7, 2024

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: 1.4.80-14
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/4606688
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:

Action Performed:

Pre-requisite: user must be logged in.

  1. Go to any chat.
  2. Tap on the compose box.
  3. Enter " test (emoji) test ".
  4. Verify the markdown is applied to the emoji on the compose box.
  5. Send the message.
  6. Verify the markdown is not applied to the emoji on the chat.

Expected Result:

The strikethrough markdown should not be applied to emojis on the compose box.

Actual Result:

The strikethrough markdown is applied to emojis on the compose box.

Workaround:

unknown

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6505433_1717783770566.Razf8050_1_.mp4

View all open jobs on GitHub

@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jun 7, 2024
Copy link

melvin-bot bot commented Jun 7, 2024

Triggered auto assignment to @mallenexpensify (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.

@kavimuru
Copy link
Author

kavimuru commented Jun 7, 2024

@mallenexpensify 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.

@kavimuru
Copy link
Author

kavimuru commented Jun 7, 2024

We think this bug might be related to 3vip-vsb

@melvin-bot melvin-bot bot added the Overdue label Jun 10, 2024
@mallenexpensify mallenexpensify changed the title Chat - Strikethrough markdown is applied to emojis on compose box [HOLD #14676] Chat - Strikethrough markdown is applied to emojis on compose box Jun 10, 2024
@mallenexpensify mallenexpensify added Needs Reproduction Reproducible steps needed Weekly KSv2 and removed Daily KSv2 labels Jun 10, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 10, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@mallenexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot added the Overdue label Jun 19, 2024
@mallenexpensify
Copy link
Contributor

#14676
Still open

@melvin-bot melvin-bot bot removed the Overdue label Jun 21, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 1, 2024
@mallenexpensify
Copy link
Contributor

holding

@melvin-bot melvin-bot bot removed the Overdue label Jul 3, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 11, 2024
@mallenexpensify mallenexpensify added Monthly KSv2 and removed Weekly KSv2 labels Jul 17, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jul 17, 2024
@mallenexpensify
Copy link
Contributor

Bumped to monthly cuz the issue we're held on isn't a priority #14676

@mallenexpensify
Copy link
Contributor

@melvin-bot melvin-bot bot removed the Overdue label Aug 21, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 23, 2024
@mallenexpensify
Copy link
Contributor

Checking back ~monthly on the help issue

@melvin-bot melvin-bot bot removed the Overdue label Sep 24, 2024
@IuliiaHerets
Copy link

@mallenexpensify QA team found a similar issue, but the emoji is strikethrough after it had been sent too.
Do we need to raise a new issue or is it the same as above?

Strikethrough.markdown.mp4

@mallenexpensify
Copy link
Contributor

No need for a separate issue @IuliiaHerets , best to keep in this one (for if/when we prioritize). Thx

@mallenexpensify
Copy link
Contributor

Off the next week, in case this goes overdue

@melvin-bot melvin-bot bot added the Overdue label Dec 9, 2024
@mallenexpensify
Copy link
Contributor

Below is the PR from the issue we're held on, it's under review

@melvin-bot melvin-bot bot removed the Overdue label Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Monthly KSv2 Needs Reproduction Reproducible steps needed
Projects
None yet
Development

No branches or pull requests

4 participants