Skip to content

Remove markdown from comment search result

Changzheng Liu requested to merge 292435_unescape_trucated_search_result into master

What does this MR do?

This is related #292435 (closed)

I took the solution 2 mentioned in the original issue, which was to remove the markdown.

  1. Don't render markdown, remove all the markdown rendering and fix the escaping problem which is probably related to this

Screenshots (strongly suggested)

Before the change

before_change

After the change (Basic Search)

after_change

After the change (Advanced Search)

Screen_Shot_2021-03-17_at_8.32.35_PM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Changzheng Liu

Merge request reports