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

[betafix] Corrige la pagination du nombre de messages d'un utilisateur #4443

Merged
merged 1 commit into from Aug 4, 2017

Conversation

Projects
None yet
3 participants
@pierre-24
Member

pierre-24 commented Aug 4, 2017

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

Note: j'ai pas regardé les tests, j'attend Travis :p

QA

  • Créer un forum caché via l'adminstration, avec accès pour le staff
  • Avec admin, créer un topic dans ce forum caché et un topic dans un forum pas caché. Répondez également à un troisième puis un 4ième topic.
  • settings.ZDS_APP['forum']['posts_per_page'] = 2
  • Avec user, rendez vous sur la page de profil de admin. Constatez qu'il y a deux topics et deux messages. Allez voir le détail et constatez qu'il est indiqué qu'un message est inaccessible, et un post inaccessible. Constatez que la pagination fonctionne.
  • Avec admin, message et topic cachés devraient apparaitre, et l'indication disparaitre
  • Code review
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 4, 2017

Coverage Status

Coverage increased (+0.004%) to 89.076% when pulling 5d50b66 on pierre-24:betafix_messages_of_user into 5656ea2 on zestedesavoir:release-v24.

coveralls commented Aug 4, 2017

Coverage Status

Coverage increased (+0.004%) to 89.076% when pulling 5d50b66 on pierre-24:betafix_messages_of_user into 5656ea2 on zestedesavoir:release-v24.

@pierre-24 pierre-24 added this to the v24 milestone Aug 4, 2017

@vhf vhf added the QA svp label Aug 4, 2017

@vhf

This comment has been minimized.

Show comment
Hide comment
@vhf

vhf Aug 4, 2017

Member

QA OK

Member

vhf commented Aug 4, 2017

QA OK

@vhf vhf removed the QA svp label Aug 4, 2017

@vhf vhf merged commit ee2e926 into zestedesavoir:release-v24 Aug 4, 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