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

Jump to bottom

Les descriptions vides sont affichées pour les contenus #5445

Closed
nils-van-zuijlen opened this issue Sep 18, 2019 · 3 comments
Closed

Les descriptions vides sont affichées pour les contenus #5445

nils-van-zuijlen opened this issue Sep 18, 2019 · 3 comments
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant

Comments

@nils-van-zuijlen
Copy link
Contributor

Les contenus ayant une description vide se voient afficher None sur la page d’accueil et dans la bibliothèque.

C'est sans doute une conversion depuis la base de données qui a changé.

Sujet : https://zestedesavoir.com/forums/sujet/13013/les-descriptions-vides-sont-affichees-pour-les-contenus/
Envoyé depuis Zeste de Savoir
@Situphen Situphen added the S-Régression Corrige un problème sur un composant qui fonctionnait auparavant label Sep 18, 2019
@Situphen
Copy link
Member

Très probablement une régression de #5364 qui a modifié la fonction description(self) de la classe PublishedContent dans le fichier zds/tutorialv2/models/database.py, fonction qui est utilisée pour afficher le sous-titre dans les vignettes des contenus.

@Situphen Situphen added the C-Back Concerne le back-end Django label Sep 18, 2019
@A-312
Copy link
Contributor

A-312 commented Sep 19, 2019 •
edited

Vous pouvez vérifier si le logo s'affiche correctement sur les tweets aussi ? J'ai testé sur mon portable appartement, ils n'ont pas d'icône. EDIT: OK

@artragis artragis added the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Sep 19, 2019
@A-312
Copy link
Contributor

A-312 commented Sep 19, 2019

La PR #5364 fonctionne pour les sujets sur le forum mais pas pour les contenus. :(

Situphen added a commit to Situphen/zds-site that referenced this issue Sep 19, 2019
@Situphen
L'absence de sous-titre n'est plus remplacée par None (fix zestedesav…
f2ab059 
…oir#5445)
@Situphen Situphen mentioned this issue Sep 19, 2019
Merged
Situphen added a commit that referenced this issue Sep 21, 2019
@Situphen
L'absence de sous-titre n'est plus remplacée par None (fix #5445)
3f28efa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@artragis @Situphen @A-312 @nils-van-zuijlen