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

Permet de verrouiller les commentaires d'un contenu #6598

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Situphen
Copy link
Member

Fixes #6563

Une partie de la fonctionnalité est déjà codée (content.is_locked sur la même base que topic.is_locked) mais il manquait quelques bouts que j'ai rajouté pour avoir quelque chose de fonctionnel. Je me suis inspiré de bouts de code existants donc j'espère que c'est quand même assez propre au final. Je veux bien vos retours sur le code si possible !

Je met l'étiquette Bloquant car c'est une fonctionnalité assez demandée par les modérateurs vu tout le spam que l'on a dans les commentaires.

@Situphen Situphen added the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Mar 30, 2024
@Situphen Situphen added this to En développement in Suivi des PR via automation Mar 30, 2024
@coveralls
Copy link

Coverage Status

coverage: 88.645% (-0.05%) from 88.699%
when pulling 7e8ee01 on Situphen:locked-comments
into 6e0d801 on zestedesavoir:dev.

@philippemilink
Copy link
Member

J'ai parcouru le code rapidement, ça me semble bien. J'ai juste une remarque : renommer is_locked en is_content_reaction_locked (ou quelque chose d'autre) pour que ce soit explicite que c'est les commentaires du contenu qu'on souhaite verrouiller (on pourrait imaginer dans le futur qu'on puisse verrouiller un contenu, quelque soit le sens qu'on mette derrière).

Et il manque les tests :)

@Arnaud-D
Copy link
Contributor

"content_reaction" n'est pas forcément un très bon nom non plus (ce sont des commentaires plus que des réactions). Après, c'est peut-être hors périmètre de cette PR de changer ça.

@philippemilink
Copy link
Member

"content_reaction" n'est pas forcément un très bon nom non plus (ce sont des commentaires plus que des réactions).

Je suis bien d'accord, mais il me semble que c'est comme ça que sont nommés les commentaires de contenu dans le code.

Après, c'est peut-être hors périmètre de cette PR de changer ça.

Bien d'accord.

@Arnaud-D
Copy link
Contributor

Oui, si jamais on changeait ce nom, ça serait partout. ^^

@Situphen
Copy link
Member Author

@philippemilink J'ai bien en tête le renommage et les tests, mais il manque d'autres choses. Je vous en parlerai lors de la prochaine réunion mais j'ai ouvert une boîte de pandore car les commentaires c'est un méli-mélo du module des contenus et de celui du forum, donc il y a un peu de refacto en perspective si on veut bien faire les choses.

@Situphen Situphen removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Suivi des PR
  
En développement
Development

Successfully merging this pull request may close these issues.

Permettre de fermer la zone de commentaires d'un contenu
4 participants