Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(documentation): ajouter des vidéos YT dans les fiches pratiques #597

Merged
merged 8 commits into from
Apr 18, 2024

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Apr 15, 2024

Description

🎸 Permettre l'ajout de vidéos YT dans les fiches pratiques

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).
🚧 technique

Points d'attention

🦺 Désactiver les CSP en mode débug, pour tester en local
🦺 Autoriser le rendu de code html lors de la sauvegarde d'une instance de type Forum : safe_mode=False dans MACHINA_MARKUP_LANGAGE
🦺 Les codes html restent interdits dans les messages (Post) des utilisateurs : ajout de safe_mode=True dans la fonction de rendering de la méthode check_post_approbation
🦺 Ajout d'une div avec la classe textarea_cms_md pour gérer le dimensionnement de l'iframe
⚠️ ⚠️ encapsulation de chaque iframe dans une balise div pour ne pas casser le rendu de la page. L'ajout est fait lors du rendu de forum.description.rendered pendant l'enregistrement de Forum. En cas d'édition, les balises div seront vraisemblablement présentes : pas d'ajout supplémentaire ;)

Captures d'écran (optionnel)

image

@vincentporte vincentporte added the recette-jetable review_app label Apr 15, 2024
@vincentporte vincentporte self-assigned this Apr 15, 2024
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@vincentporte vincentporte removed the recette-jetable review_app label Apr 15, 2024
@vincentporte vincentporte force-pushed the vincentporte/yt_firsttry branch from 06a9c9f to f8bfd68 Compare April 16, 2024 12:46
@vincentporte vincentporte marked this pull request as ready for review April 16, 2024 12:50
@vincentporte vincentporte changed the title yt 1st try [DOCUMENTATION] ajouter des vidéos YT dans les fiches pratiques Apr 16, 2024
config/settings/base.py Outdated Show resolved Hide resolved
@vincentporte vincentporte force-pushed the vincentporte/yt_firsttry branch from f8bfd68 to d99d1f7 Compare April 16, 2024 13:05
@vincentporte vincentporte added the recette-jetable review_app label Apr 16, 2024
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@vincentporte vincentporte linked an issue Apr 16, 2024 that may be closed by this pull request
Copy link
Contributor

@hellodeloo hellodeloo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vincentporte vincentporte force-pushed the vincentporte/yt_firsttry branch from d99d1f7 to f6d73a1 Compare April 16, 2024 13:49
@vincentporte vincentporte force-pushed the vincentporte/yt_firsttry branch 2 times, most recently from b3abfc9 to c69d84b Compare April 17, 2024 13:01
@vincentporte vincentporte force-pushed the vincentporte/yt_firsttry branch from c69d84b to 158ac51 Compare April 17, 2024 13:04
@vincentporte vincentporte changed the title [DOCUMENTATION] ajouter des vidéos YT dans les fiches pratiques feat(documentation): ajouter des vidéos YT dans les fiches pratiques Apr 18, 2024
@vincentporte vincentporte merged commit d666a0b into master Apr 18, 2024
4 of 5 checks passed
@vincentporte vincentporte deleted the vincentporte/yt_firsttry branch April 18, 2024 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[VIDEO] embed player
3 participants