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

Rend le boutton 'cette réponse m'a aidé' disponible sur les message de l'OP #3871

Merged
merged 1 commit into from
Oct 19, 2016
Merged

Rend le boutton 'cette réponse m'a aidé' disponible sur les message de l'OP #3871

merged 1 commit into from
Oct 19, 2016

Conversation

AmarOk1412
Copy link
Member

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) #3867

QA

  • Vérifier qu'un OP peut cliquer sur "cette réponse m'a aidé" sur tous les messages de son topic
  • Vérifier qu'un utilisateur qui n'est pas auteur ne peut pas cliquer sur ce boutton
  • Vu qu'il s'agit surtout de la suppression d'un peu de code, il faut vérifier que ça ne casse pas d'autres tests

@coveralls
Copy link

coveralls commented Oct 19, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 87.609% when pulling 197c3ea on AmarOk1412:issue_3867 into bd34531 on zestedesavoir:dev.

@pierre-24
Copy link
Member

QA OK, on est parti :)

@pierre-24 pierre-24 merged commit 624c0f3 into zestedesavoir:dev Oct 19, 2016
@pierre-24 pierre-24 added this to the Version de développement milestone Oct 19, 2016
@AmarOk1412 AmarOk1412 deleted the issue_3867 branch October 19, 2016 19:15
@vhf vhf added S-Évolution C-Front Concerne l'interface du site C-Back Concerne le back-end Django labels Oct 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vhf vhf vhf approved these changes

Assignees
No one assigned
Labels
C-Back Concerne le back-end Django C-Front Concerne l'interface du site
Projects
None yet
Milestone
V21
Development

Successfully merging this pull request may close these issues.
4 participants
@AmarOk1412 @coveralls @pierre-24 @vhf