Skip to content
This repository was archived by the owner on Apr 6, 2022. It is now read-only.

markdown in comments #24

Merged
merged 10 commits into from
May 3, 2018
Merged

markdown in comments #24

merged 10 commits into from
May 3, 2018

Conversation

patte
Copy link
Contributor

@patte patte commented Apr 11, 2018 •
edited
Loading

resolves: #23
TODOs

  • mdast content in Comment
  • render markdown in email
  • sort: chronological without bubble
  • change cf_comment_notification_new_mdast back to cf_comment_notification_new

@coveralls
Copy link

coveralls commented Apr 11, 2018 •
edited
Loading

Pull Request Test Coverage Report for Build 128

  • 0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.965%
Totals Coverage Status
Change from base Build 126: 0.0%
Covered Lines: 44
Relevant Lines: 47
💛 - Coveralls

@uxengine uxengine mentioned this pull request Apr 26, 2018
Merged
@patte patte changed the title WIP: markdown in comments markdown in comments May 2, 2018
@patte patte temporarily deployed to republik-api-staging May 2, 2018 09:36 Inactive
@patte patte temporarily deployed to republik-api-staging May 2, 2018 09:42 Inactive
@patte patte temporarily deployed to republik-api-staging May 2, 2018 09:55 Inactive
@patte patte temporarily deployed to republik-api-staging May 3, 2018 09:15 Inactive
@patte patte merged commit 3d72ab5 into master May 3, 2018
@patte patte deleted the comments-mdast branch May 3, 2018 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

support markdown in comments
3 participants
@patte @coveralls @uxengine