Add markdown support for summary #482

Merged
Plume_migration_agent merged 4 commits from summary-md into master 2019-03-17 19:11:29 +00:00

I should probably add a new field to keep raw md
Fixes #247

I should probably add a new field to keep raw md Fixes #247
elegaanz commented 2019-03-16 15:11:48 +00:00 (Migrated from github.com)

Also once #460 is merged, it would be interesting to use the "inline" mode for Markdown that was introduced, to avoid having profiles with big images or titles in their summaries.

Also once #460 is merged, it would be interesting to use the "inline" mode for Markdown that was introduced, to avoid having profiles with big images or titles in their summaries.
codecov[bot] commented 2019-03-16 15:12:40 +00:00 (Migrated from github.com)

Codecov Report

Merging #482 into master will increase coverage by 0.19%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #482      +/-   ##
==========================================
+ Coverage   26.82%   27.01%   +0.19%     
==========================================
  Files          64       64              
  Lines        7423     7432       +9     
==========================================
+ Hits         1991     2008      +17     
+ Misses       5432     5424       -8
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/482?src=pr&el=h1) Report > Merging [#482](https://codecov.io/gh/Plume-org/Plume/pull/482?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/a2b2e37aa0099aac451067ebf4526aa82f5eca51?src=pr&el=desc) will **increase** coverage by `0.19%`. > The diff coverage is `57.14%`. ```diff @@ Coverage Diff @@ ## master #482 +/- ## ========================================== + Coverage 26.82% 27.01% +0.19% ========================================== Files 64 64 Lines 7423 7432 +9 ========================================== + Hits 1991 2008 +17 + Misses 5432 5424 -8 ```
elegaanz (Migrated from github.com) reviewed 2019-03-16 15:57:19 +00:00
elegaanz (Migrated from github.com) left a comment

👌

:ok_hand:
elegaanz (Migrated from github.com) approved these changes 2019-03-16 16:41:53 +00:00
elegaanz (Migrated from github.com) left a comment

Even more 👌 now that the tests work.

Even more :ok_hand: now that the tests work.
igalic commented 2019-03-16 17:34:42 +00:00 (Migrated from github.com)

idea for testing migrations: run the setup, then run redo


would make a lot more sense of we had done test data, too 😅

idea for testing migrations: run the setup, then run `redo` ----- would make a lot more sense of we had done test data, too 😅
Author
Owner

I don't know how to do that in Travis, but it's actually what I'm doing on my computer

I don't know how to do that in Travis, but it's actually what I'm doing on my computer
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#482
No description provided.