Add markdown support for summary #482

Yhdistetty

Plume_migration_agent yhdistetty 4 committia lähteestä summary-md kohteeseen master

trinity-1686a kommentoi

Omistaja

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

👍 1

elegaanz kommentoi (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.

👍 1

codecov[bot] kommentoi (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

elegaanz (Migrated from github.com) jätti kommentin

👌

:ok_hand:

elegaanz (Migrated from github.com) hyväksyi nämä muutokset

elegaanz (Migrated from github.com) jätti kommentin

Even more 👌 now that the tests work.

Even more :ok_hand: now that the tests work.

igalic kommentoi (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 😅

trinity-1686a kommentoi

Tekijä

Omistaja

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

Reviewers

The pull request has been merged as 570d7fe2d0.

You can also view command line instructions.

Vaihe 1:

From your project repository, check out a new branch and test the changes.

git checkout -b summary-md master

git pull origin summary-md

Vaihe 2:

Merge the changes and update on Forgejo.

git checkout master

git merge --no-ff summary-md

git push origin master

No reviewers

Ei merkkipaalua

Ei käsittelijää

2 osallistujaa

Ilmoitukset

Määräpäivä

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

Määräpäivää ei asetettu.

Riippuvuudet

Riippuvuuksia ei asetettu.

Ladataan…

Sisältöä ei vielä ole.

Tunnisteet Merkkipaalut

Add markdown support for summary #482

Codecov Report

Reviewers

Vaihe 1:

Vaihe 2: