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

Ajout des likes/dislikes dans les messages privés #6317

Merged
merged 3 commits into from
Jun 16, 2022

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented May 22, 2022

Fix #6119.

Porte le système de likes/dislikes aux messages privés. J'ai réutilisé une partie des composantes existantes (en particulier niveau front-end) mais j'ai aussi créé des nouvelles classes en essayant de ne pas rendre le tout trop confusant.

Contrôle qualité

Scénario 1:

  • Se connecter en tant que User1
  • Envoyer un MP à User2
  • Essayer de cliquer sur le pouce vers le haut ou vers le bas
    Résultat attendu: Aucun effet: il est impossible d'utiliser les likes/dislikes sur ses propres messages

Scénario 2:

  • Se connecter en tant que User1
  • Envoyer un MP à User2
  • Se connecter en tant que User2
  • Ouvrir le MP et cliquer sur le pouce vers le haut
    Résultat attendu: Le vote est bien enregistré
  • Passer la souris au dessus du pouce vers le haut
    Résultat attendu: Le nom des voteurs (ici, User2) est affiché à côté du curseur

Scénario 3: Faire le même test avec le pouce vers le bas

Scénario 4:

  • Se connecter en tant que User1
  • Envoyer un MP à User2
  • Se connecter en tant que User2
  • Ouvrir le MP et cliquer sur le pouce vers le haut
  • Cliquer une seconde fois sur le pouce vers le haut
    Résultat attendu: Le vote est annulé

Scénario 5: Faire le même test avec le pouce vers le bas

@coveralls
Copy link

coveralls commented May 25, 2022

Coverage Status

Coverage increased (+0.03%) to 87.91% when pulling 0311b9f on Migwel:issue6119 into ec43b7e on zestedesavoir:dev.

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.

QA OK ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) June 16, 2022 16:57
@Arnaud-D Arnaud-D merged commit 7b8d466 into zestedesavoir:dev Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Garder les ±1 dans les MPs
3 participants