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

Empêche d’éditer un message masqué #3826

Merged
merged 3 commits into from
Sep 7, 2016
Merged

Empêche d’éditer un message masqué #3826

merged 3 commits into from
Sep 7, 2016

Conversation

Karnaj
Copy link
Contributor

@Karnaj Karnaj commented Aug 30, 2016

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

QA

  • Créer un topic avec un membre non staff.
  • Masquer le premier message.
  • Vérifier que le bouton « Éditer le sujet » n’apparaît plus.
  • Vérifier qu’on ne peut pas éditer le message en passant par l’url forums/message/editer/?message={pk_mesasage}.
  • Vérifier qu’on ne peut pas éditer le sujet en passant par l’url forums/sujet/editer/?topic={pk_topic}.
  • Vérifier qu’un membre staff a toujours accès au bouton « Éditer le sujet » et qu’il peut éditer le sujet et le message.

@Karnaj Karnaj changed the title Fix 3804 Empêche d’éditer un message masqué Aug 30, 2016
@coveralls
Copy link

coveralls commented Aug 30, 2016

Coverage Status

Coverage increased (+0.003%) to 87.738% when pulling e825076 on Karnaj:fix-3804 into c1b736d on zestedesavoir:release-v20.

@vhf vhf added this to the Version 20 milestone Sep 7, 2016
@vhf vhf added the C-Back Concerne le back-end Django label Sep 7, 2016
@vhf
Copy link
Contributor

vhf commented Sep 7, 2016

QA ok. Merci !

@vhf vhf merged commit 8ca9dfe into zestedesavoir:release-v20 Sep 7, 2016
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants