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.

Sign up for GitHub

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

Compte les commentaires non masqués #6156

Merged
merged 2 commits into from
Sep 11, 2021
Merged

Compte les commentaires non masqués #6156

merged 2 commits into from
Sep 11, 2021

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Sep 3, 2021

Cette PR permet de compter uniquement les messages non masqués sur les commentaires de contenus.

Numéro du ticket concerné : #6145

Contrôle qualité

  • Demarrez le site make run-back et assurez vous d'avoir généré des fixtures generate-fixtures
  • Connectez vous avec l'utilisateur admin
  • Allez dans des commentaires de contenus (billet, article, tuto) et masquez un commentaire
  • Assurez vous que sur la page d'accueil, la page de la bibliothèque et la page de la tribune, que le nombre de commentaire affiché dans la bulle adéquate correspond au nombre de commentaires non masqués.

@firm1 firm1 added the C-Back Concerne le back-end Django label Sep 3, 2021
@coveralls
Copy link

coveralls commented Sep 3, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 86.621% when pulling 8cc5946 on firm1:fix6145 into 91b28d6 on zestedesavoir:dev.

philippemilink
philippemilink approved these changes Sep 11, 2021
View reviewed changes
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.

Rapport de QA

OK ✔️ (testé avec en masquant un commentaire dans un billet)

@philippemilink philippemilink enabled auto-merge (squash) September 11, 2021 16:51
@philippemilink philippemilink merged commit 77b8c63 into zestedesavoir:dev Sep 11, 2021
@philippemilink philippemilink mentioned this pull request Sep 11, 2021
Closed
This was referenced Sep 13, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Back Concerne le back-end Django
Projects
Suivi des PR
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@firm1 @coveralls @philippemilink