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

Billets précédents et suivants : uniquement ceux choisis par le Staff #5793

Closed
Situphen opened this issue May 19, 2020 · 4 comments · Fixed by #6443
Closed

Billets précédents et suivants : uniquement ceux choisis par le Staff #5793

Situphen opened this issue May 19, 2020 · 4 comments · Fixed by #6443
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement ! S-Évolution Ajoute de nouvelles fonctionnalités

Comments

@Situphen
Copy link
Member

Discussion sur le forum

Les billets écrits ne sont pas validés avant publication par le Staff, n’importe quel membre peut publier son billet quand il le souhaite. Par contre, les billets affichés sur la page d’accueil sont ceux sélectionnés par le Staff, ce qui n’a pas été le cas de "L’hygiènisme une bombe à retardement" pour l’instant.

Il serait à mon avis souhaitable et cohérent que les liens Précédent et Suivant en bas des billets ne proposent que les billets sélectionnés par le Staff.

Il faut ajouter dans le fichier zds/tutorialv2/views/published.py quelque chose comme :

if self.current_content_type == 'OPINION':
    queryset_pagination = queryset_pagination.filter(content__sha_picked=F('sha_public'))

au niveau des ces lignes :

if self.current_content_type in ('ARTICLE', 'OPINION'):
queryset_pagination = PublishedContent.objects.filter(content_type=self.current_content_type,
must_redirect=False)
context['previous_content'] = queryset_pagination \
.filter(publication_date__lt=self.public_content_object.publication_date) \
.order_by('-publication_date').first()
context['next_content'] = queryset_pagination \
.filter(publication_date__gt=self.public_content_object.publication_date) \
.order_by('publication_date').first()

Il faut ajouter au tout début du fichier from django.db.models import F.

@Situphen Situphen added C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités labels May 19, 2020
@Situphen Situphen added the Facile Bon ticket pour débuter pour rejoindre le développement ! label May 19, 2020
@leroivi
Copy link
Contributor

leroivi commented Jul 17, 2020

Je prends ;)

HSleymn added a commit to HSleymn/zds-site that referenced this issue Feb 18, 2021
Etant débutant, j'ai essayé de régler ce problème. zestedesavoir#5793
@HSleymn HSleymn mentioned this issue Feb 18, 2021
@HSleymn
Copy link

HSleymn commented Feb 18, 2021

alors j'ai essayé de le régler, et si c'est bon, c'est réglé alors !

@AmauryCarrade AmauryCarrade linked a pull request Feb 18, 2021 that will close this issue
HSleymn added a commit to HSleymn/zds-site that referenced this issue Feb 18, 2021
Ajout faisant en sorte que les boutons Précédents et Suivants dans la bibliothèque de billets, proposent que les billets validé par le Staff.
Lignes ajoutées : 88-90
Issue lien : zestedesavoir#5793
@AmauryCarrade AmauryCarrade removed a link to a pull request Feb 18, 2021
@leroivi
Copy link
Contributor

leroivi commented Feb 20, 2021

Attention au test_opinion_link_is_not_related_to_the_author, il est à mettre à jour en conséquence

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Oct 1, 2022

Pour info, ce travail est en attente de reprise, donc n'importe qui peut reprendre ce sujet, quitte à s'inspirer du travail déjà effectué.

@Arnaud-D Arnaud-D moved this to À trier in Suivi des tickets Dec 17, 2022
@philippemilink philippemilink moved this from À trier to À traiter in Suivi des tickets Dec 18, 2022
atman0eirb added a commit to atman0eirb/zds-site that referenced this issue Jan 13, 2023
modify test : test_opinion_link_is_not_related_to_the_author in
 file : zds/tutorialv2/tests/tests_views/tests_published.py 
 to accept the Fix zestedesavoir#5793
@github-project-automation github-project-automation bot moved this from À traiter to Clos in Suivi des tickets Jan 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement ! S-Évolution Ajoute de nouvelles fonctionnalités
Projects
Status: Clos
4 participants