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

Empêche un utilisateur sans droits de signaler ses messages masqués #4537

Merged
merged 2 commits into from Aug 18, 2017

Conversation

Projects
None yet
4 participants
@AmarOk1412
Member

AmarOk1412 commented Aug 18, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #4379

QA

  • Créer un message avec un utilisateur et le masquer.
  • Vérifier que les utilisateurs sans droits n'ont pas la possibilité de signaler le message masqué
  • Vérifier qu'un modérateur puisse signaler et démasquer le message.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling 5f80017 on AmarOk1412:issue_4379 into 51399ee on zestedesavoir:dev.

coveralls commented Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling 5f80017 on AmarOk1412:issue_4379 into 51399ee on zestedesavoir:dev.

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Aug 18, 2017

Member

QA OK sinon. Je te laisse rebase et corriger l'indentation et je merge. :)

Member

gcodeur commented Aug 18, 2017

QA OK sinon. Je te laisse rebase et corriger l'indentation et je merge. :)

AmarOk1412 added some commits Aug 18, 2017

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling c0f5b80 on AmarOk1412:issue_4379 into 3a1ac20 on zestedesavoir:dev.

coveralls commented Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling c0f5b80 on AmarOk1412:issue_4379 into 3a1ac20 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling c0f5b80 on AmarOk1412:issue_4379 into 3a1ac20 on zestedesavoir:dev.

coveralls commented Aug 18, 2017

Coverage Status

Coverage remained the same at 89.377% when pulling c0f5b80 on AmarOk1412:issue_4379 into 3a1ac20 on zestedesavoir:dev.

@gcodeur gcodeur added this to the Version de développement milestone Aug 18, 2017

@gcodeur gcodeur merged commit 766568c into zestedesavoir:dev Aug 18, 2017

2 checks passed

codacy/pr Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment