Skip to content

[NEW] Add markdown parser "marked"#6158

Closed
ghost wants to merge 27 commits intodevelopfrom
unknown repository
Closed

[NEW] Add markdown parser "marked"#6158
ghost wants to merge 27 commits intodevelopfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Feb 24, 2017

@RocketChat/core

marked is support GFM.

  • ruler
  • tables
  • lists

ss


You can select parser Original or Marked.

ss2

@shtefcs
Copy link
Copy Markdown

shtefcs commented Mar 3, 2017

This is really cool @nishimaki10 .

Waiting for approval of this PR.

@RichardFoxworthy
Copy link
Copy Markdown

Will markdown messages be rendered properly in email alerts?

@shtefcs
Copy link
Copy Markdown

shtefcs commented Mar 15, 2017

@RichardFoxworthy Can't confirm, but I guess will not.

@engelgabriel engelgabriel modified the milestones: 0.54.0, 0.55.0 Mar 22, 2017
@engelgabriel engelgabriel modified the milestones: 0.55.0, 0.56.0 Apr 13, 2017
@engelgabriel engelgabriel modified the milestones: 0.57.0, 0.58.0 Jun 19, 2017
@rodrigok rodrigok modified the milestones: 0.58.0, 0.59.0 Aug 1, 2017
@rodrigok
Copy link
Copy Markdown
Member

@nishimaki10 can you solve the conflicts?

@RocketChat RocketChat deleted a comment Aug 22, 2017
@RocketChat RocketChat deleted a comment Aug 22, 2017
@RocketChat RocketChat deleted a comment Aug 22, 2017
@RocketChat RocketChat deleted a comment Aug 22, 2017
@RocketChat RocketChat deleted a comment Aug 22, 2017
@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 22, 2017

@rodrigok yes, solved.

@rodrigok
Copy link
Copy Markdown
Member

@nishimaki10 Idk why, but travis think this PR as an abuse, so I'll create a new one

image

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-7852 August 23, 2017 17:41 Inactive
@rodrigok
Copy link
Copy Markdown
Member

Closed in favor of #7852

@rodrigok rodrigok closed this Aug 23, 2017
@ghost ghost deleted the add-marked branch September 5, 2017 08:21
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.

5 participants