Add markdown support for summary #482

Fusionnée
Plume_migration_agent a fusionné 4 révision(s) à partir de summary-md vers master il y a 5 ans
Propriétaire

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 a commenté il y a 5 ans (Migré de 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] a commenté il y a 5 ans (Migré de 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 (Migré de github.com) révisé il y a 5 ans
elegaanz (Migré de github.com) laisser un commentaire

👌

:ok_hand:
elegaanz (Migré de github.com) ces changements ont été approuvés il y a 5 ans
elegaanz (Migré de github.com) laisser un commentaire

Even more 👌 now that the tests work.

Even more :ok_hand: now that the tests work.
igalic a commenté il y a 5 ans (Migré de 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 😅
Éditeur
Propriétaire

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

Relecteurs

La demande d'ajout a été fusionnée en 570d7fe2d0.
Vous pouvez également voir les instructions en ligne de commande.

Étape 1:

Depuis le dépôt de votre projet, sélectionnez une nouvelle branche et testez les modifications.
git checkout -b summary-md master
git pull origin summary-md

Étape 2:

Fusionner les modifications et mettre à jour sur Forgejo.
git checkout master
git merge --no-ff summary-md
git push origin master
Connectez-vous pour rejoindre cette conversation.
Aucune évaluation
Aucun jalon
Pas d'assignataires
2 participants
Notifications
Échéance
La date d’échéance est invalide ou hors plage. Veuillez utiliser le format 'aaaa-mm-dd'.

Aucune échéance n'a été définie.

Dépendances

No dependencies set.

Reference: Plume/Plume#482
Chargement…
Il n'existe pas encore de contenu.