Skip to content

[HOLD on #40845] LHN - HTML displayed in LHN when sending markdown message with attachment #41141

Closed

Description

@lanitochka17

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.67-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): abebemiherat@gmail.com
Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Log in in Gmail account
  3. Go to chat and write a message with markdown
  4. Click the plus sign and select attachment
  5. Send the attachment with the message. Notice in the LHN, the HTML is displayed

Expected Result:

When sending a markdown message with an attachment, the LHN should not display the message with HTML

Actual Result:

When sending a markdown message with an attachment, the LHN displays the message with HTML

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

Add any screenshot/video evidence

Bug6463669_1714216456821.Screen_Recording_2024-04-27_at_3.51.15_AM.mp4

View all open jobs on GitHub

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Apr 27, 2024
  2. melvin-bot commented on Apr 27, 2024

    @melvin-bot

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

  3. lanitochka17 commented on Apr 27, 2024

    @lanitochka17
    Author

    @sakluger 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

  4. lanitochka17 commented on Apr 27, 2024

    @lanitochka17
    Author

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

  5. allgandalf commented on Apr 28, 2024

    @allgandalf
    Contributor

    #40845 should fix this, @sakluger let's put this on hold :)

  6. changed the title [-]LHN - HTML displayed in LHN when sending markdown message with attachment[/-] [+][HOLD on #40845] LHN - HTML displayed in LHN when sending markdown message with attachment[/+] on Apr 29, 2024
  7. melvin-bot commented on Apr 30, 2024

    @melvin-bot

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

  8. sakluger commented on Apr 30, 2024

    @sakluger
    Contributor

    Thanks @GandalfGwaihir!

  9. added and removed on Apr 30, 2024
  10. sakluger commented on May 10, 2024

    @sakluger
    Contributor

    I tested myself and this seems to be fixed! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

BugSomething is broken. Auto assigns a BugZero manager.WeeklyKSv2

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions