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

Fix hyperlink render issue in triage reports #14

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

yli919
Copy link
Contributor

@yli919 yli919 commented Mar 28, 2024

Type of change (place an x in the [ ] that applies)

  • New sample
  • New feature
  • Bug fix
  • Documentation

Summary

For messages that have hyperlink formats like <google.com|google>, the character | was encoded as %7C which gets wrongly appended to the url. This PR is to fix the issue by replacing | with a whitespace in the message.
Before:
Screenshot 2024-03-27 at 5 05 47 PM

After:
Screenshot 2024-03-27 at 5 05 42 PM

Requirements (place an x in each [ ] that applies)

  • I’ve checked my submission against the Samples Checklist to ensure it complies with all standards
  • I have ensured the changes I am contributing align with existing patterns and have tested and linted my code
  • I've read and agree to the Code of Conduct

@yli919 yli919 merged commit 8b56ccd into main Mar 28, 2024
2 checks passed
@yli919 yli919 deleted the yli_fix_hyperlink_in_triage_reports branch March 28, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants