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

[MM-60408] Replaced chars with HTML entities in reminder posts #409

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

raghavaggarwal2308
Copy link
Contributor

Summary

  • Replaced chars with HTML entities in reminder posts

Screenshots

Before:
image

After:
image

Ticket Link

Fixes https://mattermost.atlassian.net/browse/MM-60408

@raghavaggarwal2308 raghavaggarwal2308 self-assigned this Sep 6, 2024
@raghavaggarwal2308 raghavaggarwal2308 added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Sep 6, 2024
@raghavaggarwal2308 raghavaggarwal2308 added the 3: Security Review Review requested from Security Team label Sep 6, 2024
@raghavaggarwal2308 raghavaggarwal2308 removed 2: Dev Review Requires review by a core committer 3: Security Review Review requested from Security Team labels Sep 10, 2024
@AayushChaudhary0001
Copy link

@raghavaggarwal2308 This PR was reviewed and I found an issue regarding some markdowns
Please refer the below screenshots:-

on MS calendar
image

on MM
image

This seems to have a handling issue on Mattermost, please verify the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants