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

Fix #5022: les messages masqués ne sont plus considérés comme utiles #5023

Merged
merged 1 commit into from Aug 24, 2018
Merged

Conversation

gllmc
Copy link
Member

@gllmc gllmc commented Aug 24, 2018

Numéro du ticket concerné (optionnel) : #5022

Contrôle qualité

Vérifier que le marquage d'un post comme utile marche toujours, mais qu'un message masqué n'est plus affiché comme utile.

@gllmc gllmc added S-BUG Corrige un problème C-Front Concerne l'interface du site labels Aug 24, 2018
@coveralls
Copy link

coveralls commented Aug 24, 2018

Coverage Status

Coverage increased (+15.2%) to 89.247% when pulling ecff1ce on gcodeur:fix-5022 into 1937b86 on zestedesavoir:dev.

self.assertEqual(302, response.status_code)

response = self.client.get(topic.get_absolute_url(), follow=False)
self.assertContains(response, 'green hidden')
Copy link
Member

Choose a reason for hiding this comment

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

tu pourrais aussi ajouter un notContains("helpful")?

Copy link
Member Author

Choose a reason for hiding this comment

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

En fait non (c'était ma première intention), car il y a helpful-message dans le code, que le message soit utile ou non.

@firm1
Copy link
Contributor

firm1 commented Aug 24, 2018

Rapport qualité

QA : OK

Je merge

@firm1 firm1 merged commit 4ee284f into zestedesavoir:dev Aug 24, 2018
@gllmc gllmc deleted the fix-5022 branch August 24, 2018 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants