[ZEP-13] No more error 500.

[artragis]

J'ai corrigé l'erreur 500 qu'a vu Andr0 (https://sentry.sandhose.fr/zeste-de-savoir/beta-backend/issues/809/)

J'ai aussi tenté de comprendre pourquoi les tribunes n'apparaissent pas dans les tags.

[coveralls]

Coverage increased (+0.006%) to 87.049% when pulling d657542 on artragis:some_things_on_tribunes into ef3d824 on zestedesavoir:zep-13-b.

[coveralls]

Coverage increased (+0.007%) to 87.049% when pulling c7a8b40 on artragis:some_things_on_tribunes into ef3d824 on zestedesavoir:zep-13-b.

[vhf]

C'est un revert de 66f0093.

La question est : les billets qui n'ont pas été choisis par l'équipe de ZdS doivent-ils apparaitre dans les listes de contenus aux côtés des articles et tutoriels, c'est à dire typiquement ici ou seulement dans les listes de contenus de même type afin de ne pas les mélanger avec les articles et tutoriels, typiquement comme ici ?

Le staff semble vouloir la 2e option, les membres du forum aussi, les devs semblent préférer la première. @FanJiyong un avis ?

[pierre-24]

On mélange pas, évidement ! (et je suis dev :p )

[artragis]

ça a été fixé.

[SpaceFox]

Fonctionnellement, je ne vois pas ce qui justifie de mélanger du contenu validé et non-validé (si ce n'est que c'est plus simple à développer). Le 2 janvier 2017 à 22:48, victor felder <notifications@github.com> a écrit :

…

***@***.**** requested changes on this pull request. ------------------------------ In zds/tutorialv2/views/views_published.py <#4107 (review)> : > @@ -324,8 +324,6 @@ def get_queryset(self): if self.current_content_type: queryset = queryset.filter(content_type=self.current_content_type) - else: - queryset = queryset.filter(content__sha_picked=F('sha_public')) C'est un revert de 66f0093 <66f0093> . La question est : les billets qui n'ont pas été choisis par l'équipe de ZdS doivent-ils apparaitre dans les listes de contenus aux côtés des articles et tutoriels, c'est à dire typiquement ici <https://zestedesavoir.com/contenus/?category=programmation-et-algorithmique> ou seulement dans les listes de contenus de même type afin de ne pas les mélanger avec les articles et tutoriels, typiquement comme ici <https://zestedesavoir.com/articles/?category=programmation-et-algorithmique> ? Le staff semble vouloir la 2e option, les membres du forum aussi, les devs semblent préférer la première. @FanJiyong <https://github.com/FanJiyong> un avis ? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#4107 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFhKnEycF1Kmap8l79z-I0gDGHhuCFriks5rOXDHgaJpZM4LZMCT> .

[artragis]

Ok, donc je changerai le comportement. Mais on va faire les choses un poil plus proprement. Le 03/01/2017 à 08:08, Pierre Beaujean a écrit :

…

***@***.**** commented on this pull request. ------------------------------------------------------------------------ In zds/tutorialv2/views/views_published.py <#4107>: > @@ -324,8 +324,6 @@ def get_queryset(self): if self.current_content_type: queryset = queryset.filter(content_type=self.current_content_type) - else: - queryset = queryset.filter(content__sha_picked=F('sha_public')) On mélange pas, évidement ! (et je suis dev :p ) — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#4107>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABc_xaCl4RJ-B85G5BWUMLtq6T1bX03jks5rOfP8gaJpZM4LZMCT>.

[coveralls]

Coverage increased (+0.009%) to 87.051% when pulling 2e303c6 on artragis:some_things_on_tribunes into ef3d824 on zestedesavoir:zep-13-b.

[coveralls]

Coverage increased (+0.0009%) to 87.043% when pulling 43476d0 on artragis:some_things_on_tribunes into ef3d824 on zestedesavoir:zep-13-b.

[coveralls]

Coverage decreased (-32.0%) to 55.057% when pulling 43476d0 on artragis:some_things_on_tribunes into ef3d824 on zestedesavoir:zep-13-b.

[pierre-24]

@artragis : est ce que c'est bon avec #4114 ?

[artragis]

go to #4160