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

Corrige l'affichage des suggestions quand certaines ne sont pas publiques #6439

Merged
merged 4 commits into from
Jan 15, 2023

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Dec 20, 2022

Fix #6437.

Ce que j'ai fait :

  • ajouter les suggestions à l'interface d'admin ;
  • filtrer les suggestions pour ne garder que celles publiques à l'affichage ;
  • fais un tout petit peu de ménage au passage.

Ce que je n'ai pas fait par rapport à la discussion du ticket :

  • supprimer les suggestions à la dépublication, parce que je pense que les gains (une liste propre) sont faibles face aux risques (suppression irréversible du travail manuel d'éditorialisation en cas de dépublication temporaire) et que ça ne vaut pas le coup ;
  • corriger le template : le comportement bizarre où une suggestion sans version publique abouti à afficher le contenu lui-même ne ne se manifeste plus, mais il y a tout de même une bizarrerie dans les templates.

Contrôle qualité

  • Appliquer les migrations
  • Se connecter en tant qu'admin
  • Ajouter 2 suggestions à une publication publique (depuis l'interface d'admin ou depuis l'interface du site si vous pouvez faire fonctionner ElasticSearch).
  • Constater que les deux publications sont affichées dans l'encart.
  • Faire en sorte qu'une des deux ne soit plus publique, mais existe toujours.
  • Constater qu'une seule est affichée dans l'encart et surtout qu'il n'y a pas n'importe quoi d'affiché à la place.

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Dec 20, 2022
@Arnaud-D Arnaud-D changed the title Suggestions Corrige l'affichage des suggestions quand certaines ne sont pas publiques Dec 20, 2022
@coveralls
Copy link

coveralls commented Dec 21, 2022

Coverage Status

Coverage: 88.302% (+0.006%) from 88.296% when pulling d95227d on Arnaud-D:suggestions into 6d62a7b on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

(en plus y a des tests en plus 😍 )

@philippemilink philippemilink enabled auto-merge (squash) January 15, 2023 15:58
@philippemilink philippemilink merged commit fff03bd into zestedesavoir:dev Jan 15, 2023
@Arnaud-D Arnaud-D deleted the suggestions branch January 15, 2023 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Présence de suggestions indues dans « Ces contenus pourraient vous intéresser »
3 participants