Skip to content

Use content editor's serializer for copy_as_gfm

Himanshu Kapoor requested to merge 413902-copy-as-gfm into master

What does this MR do and why?

Use content editor's serializer for copy_as_gfm

Until now, copy_as_gfm feature used its own serializer to copy content as markdown. Since content editor also has its own serializer, we remove the serializer in use in copy_as_gfm and use the one in content editor instead.

Changelog: changed

Further context

The code in ~/content_editor/extensions and ~/behaviors/markdown is pretty much the same. This MR copies over some minor differences to the former and deletes the latter. There are not many new tests because there isn't any new functionality added. The tests are already covered by spec/features/markdown/copy_as_gfm_spec.rb.

Screenshots or screen recordings

There are no visible changes.

How to set up and validate locally

  1. Copy some content from an issue description or a comment.
  2. Paste in plain or rich text editor. It should paste it as markdown correctly.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #413902 (closed)

Edited by Himanshu Kapoor

Merge request reports