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

Résout le bug de notification lors du déplacement de sujet #3786

Merged
merged 3 commits into from Aug 23, 2016

Conversation

@vhf
Copy link
Contributor

commented Aug 23, 2016

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

Cette PR est un remake de #3784 , dont j'ai repris l'intégralité du code et le contenu de ce message.

QA

  • Suivre les instructions update.md, vérifier que tout s'est bien passé.

  • Avec un utilisateur X, suivez un forum A et B.
  • Avec un utilisateur Y, créez un sujet dans le forum A.
  • Avec un staff, déplacez le sujet du forum A vers le forum B.
  • Avec l'utilisateur X, consultez la notification du sujet et constatez que c'est marqué comme lu.
  • Avec l'utilisateur Y, créez un sujet dans le forum A.
  • Avec le staff, déplacez le sujet du forum A vers le forum C.
  • Avec l'utilisateur X, consultez la notification du sujet et constatez que c'est marqué comme lu.
  • Avec un admin, regardez les deux notifications dans l'admin.
  • Constatez que la première notification est vivante (is_dead=False).
  • Constatez que la seconde notification est morte (is_dead=True).

@vhf vhf force-pushed the vhf:fix_3762 branch from 3b053db to c7a992a Aug 23, 2016

@vhf vhf force-pushed the vhf:fix_3762 branch from c7a992a to e7b002e Aug 23, 2016

@coveralls

This comment has been minimized.

Copy link

commented Aug 23, 2016

Coverage Status

Coverage decreased (-0.06%) to 87.691% when pulling a1f0e2f on vhf:fix_3762 into cdb3ade on zestedesavoir:dev.

@A-312

This comment has been minimized.

Copy link
Member

commented Aug 23, 2016

QA ok

@GerardPaligot GerardPaligot merged commit 6bb2f75 into zestedesavoir:dev Aug 23, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@vhf

This comment has been minimized.

Copy link
Contributor Author

commented Aug 23, 2016

Merci beaucoup @A-312 !

Pour info, j'ai mis cette PR sur la beta, j'ai laissé A-312 suivre la procédure tout en m'occupant moi-même de déplacer le sujet et monitorer les notifs depuis le shell django. C'est tout en ordre.

Merci @GerardPaligot ! :)

Avec ça je lance la release, ce sera en beta et testable dans quelques minutes.

@vhf vhf deleted the vhf:fix_3762 branch Aug 23, 2016

@vhf vhf referenced this pull request Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.