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

N'envoie pas de MP lors de la dépublication de contenu sans auteur inscrit #6356

Conversation

philippemilink
Copy link
Member

Fix #6353

Contrôle qualité

Le scénario qui pose problème :

  1. Avec un utilisateur A, créer et publier un billet
  2. En tant qu'utilisateur A, se désinscrire (Paramètres > Désinscription)
  3. Se connecter en tant qu'admin et modérer ce billet (dans la section Administration de la colonne à gauche du billet)
  4. On revient sur la liste des billets et tout fonctionne comme attendu

Vérifier que le comportement normal fonctionne toujours :

  1. Avec un utilisateur A, créer et publier un billet
  2. Se connecter en tant qu'admin et modérer ce billet (dans la section Administration de la colonne à gauche du billet)
  3. On revient sur la liste des billets et tout fonctionne comme attendu
  4. Se connecter avec l'utilisateur A, il a bien reçu un MP le notifiant de la modération et la dépublication du billet

Vérifier que ça fonctionne toujours lorsqu'il y a plusieurs auteurs :

  1. Avec un utilisateur A, créer un billet, y ajouter l'utilisateur B comme auteur, publier le billet
  2. En tant qu'utilisateur A, se désinscrire (Paramètres > Désinscription)
  3. Se connecter en tant qu'admin et modérer ce billet (dans la section Administration de la colonne à gauche du billet)
  4. On revient sur la liste des billets et tout fonctionne comme attendu
  5. Se connecter avec l'utilisateur B, il a bien reçu un MP le notifiant de la modération et la dépublication du billet

Refaire les scénarios :

  • en dépubliant le billet
  • avec un tutoriel et en dépubliant
  • avec un article et en dépubliant

@philippemilink philippemilink added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Jul 15, 2022
@coveralls
Copy link

coveralls commented Jul 15, 2022

Coverage Status

Coverage increased (+0.006%) to 88.194% when pulling 2cb3802 on philippemilink:fix-unpublish-content-unregistered-user into b4d2d08 on zestedesavoir:dev.

@philippemilink philippemilink force-pushed the fix-unpublish-content-unregistered-user branch from 0d9bbab to 5bcb662 Compare July 15, 2022 20:05
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

Je n'ai encore pas testé la fonctionnalité, mais j'ai une remarque sur l'implémentation.

zds/tutorialv2/views/validations_opinions.py Outdated Show resolved Hide resolved
@philippemilink philippemilink force-pushed the fix-unpublish-content-unregistered-user branch from 5bcb662 to 96e4dad Compare July 23, 2022 17:10
@philippemilink philippemilink force-pushed the fix-unpublish-content-unregistered-user branch from 96e4dad to 2cb3802 Compare July 24, 2022 11:43
Copy link
Member

@Situphen Situphen 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 Ça fonctionne correctement !

@Situphen Situphen requested a review from Arnaud-D July 24, 2022 12:38
@Situphen Situphen enabled auto-merge (squash) July 24, 2022 12:38
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 S-BUG Corrige un problème
Projects
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

Erreur 500 lors de la modération d'un billet dont l'auteur n'est plus incrit
4 participants