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

Abonnements aux tags sur les forums #4072

Merged
merged 4 commits into from May 25, 2017

Conversation

Projects
None yet
8 participants
@ChantyTaguan
Copy link
Contributor

ChantyTaguan commented Dec 13, 2016

Q R
Type de modification Nouvelle fonctionnalité
Ticket(s) (issue(s)) concerné(s) #1264

QA

  • Abonnez-vous à un tag
  • Vérifiez qu'un sujet créer par un autre user que vous avec ce tag vous envoie une notif
  • Vérifiez q'un sujet créé pr vous-même ne crée pas de notif
  • Désabonnez-vous
  • Vérifiez que vous ne receve plus de notif en cas de création de sujet avec ce tag
  • Vérifiez tout autre cas, tordu ou non, auquel vous pensez

@ChantyTaguan ChantyTaguan force-pushed the ChantyTaguan:feature/tag-subscription branch from a9ac1b6 to 91e9242 Dec 13, 2016

@coveralls

This comment has been minimized.

Copy link

coveralls commented Dec 13, 2016

Coverage Status

Coverage decreased (-34.8%) to 52.647% when pulling 91e9242 on ChantyTaguan:feature/tag-subscription into cffb555 on zestedesavoir:dev.

@vhf vhf self-requested a review Dec 14, 2016

@vhf

This comment has been minimized.

Copy link
Contributor

vhf commented Dec 25, 2016

@ChantyTaguan est-ce que tu pourrais rebaser sur dev stp ?

@pierre-24 pierre-24 added the QA svp label Jan 10, 2017

@ChantyTaguan ChantyTaguan force-pushed the ChantyTaguan:feature/tag-subscription branch from 91e9242 to d0ac834 Jan 11, 2017

@ChantyTaguan ChantyTaguan changed the title Feature/tag subscription Abonnements aux tags sur les forums Jan 11, 2017

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 11, 2017

Coverage Status

Coverage decreased (-32.6%) to 54.256% when pulling e8f138c on ChantyTaguan:feature/tag-subscription into ef8f4af on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 11, 2017

Coverage Status

Coverage decreased (-0.04%) to 86.795% when pulling e8f138c on ChantyTaguan:feature/tag-subscription into ef8f4af on zestedesavoir:dev.

@ChantyTaguan

This comment has been minimized.

Copy link
Contributor Author

ChantyTaguan commented Jan 11, 2017

Voili voilou ! Rebase et tests corrigés !

@pierre-24

This comment has been minimized.

Copy link
Member

pierre-24 commented Jan 12, 2017

Rapport de QA: quasiment OK, à un détail près:

screenshot from 2017-01-12 20 07 00

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 13, 2017

Coverage Status

Coverage decreased (-15.5%) to 71.32% when pulling e948733 on ChantyTaguan:feature/tag-subscription into ef8f4af on zestedesavoir:dev.

@pierre-24

This comment has been minimized.

Copy link
Member

pierre-24 commented Feb 8, 2017

Rapport de QA (3 ans après):

  • Il y a une quote incorrecte d'après le nouvel outil :)
  • Quand on s'abonne, le nombre n'est pas mis à jour, alors que c'est le cas lorsqu'on s'abonne à un sujet. J'ai l'impression que c'est une régression et que ça fonctionnait la fois passée, mais je me trompe peut être :o
@vhf

vhf approved these changes Feb 10, 2017

notifications = Notification.objects \
.filter(object_id=topic.pk, content_type__pk=topic_content_type.pk, is_read=False).all()
for notification in notifications:
is_still_valid = not notification.subscription.content_type == tag_content_type

This comment has been minimized.

Copy link
@vhf

vhf Feb 10, 2017

Contributor

Pas très lisible, le a = not b == c. Plutôt a = b != c ?

@artragis

This comment has been minimized.

Copy link
Member

artragis commented Apr 16, 2017

@ChantyTaguan tu peux reprendre le travail pour une intégration en v24 ou pas?

@ChantyTaguan ChantyTaguan force-pushed the ChantyTaguan:feature/tag-subscription branch from e948733 to 13c5045 May 11, 2017

@ChantyTaguan

This comment has been minimized.

Copy link
Contributor Author

ChantyTaguan commented May 11, 2017

@artragis Hello, je viens de faire le rebase qiu s'est étonnament très très bien passé (pas vérifié les tests par contre) :) J'essaye de voir encore cette semaine pour faire les corrections restantes.

@ChantyTaguan

This comment has been minimized.

Copy link
Contributor Author

ChantyTaguan commented May 11, 2017

J'ai un problème avec la debug_toolbar...

NoReverseMatch: u'djdt' is not a registered namespace

J'ai mis mon environnement à jour, les build passent, l'appli se lance mais ça fait kaboom quand je l'ouvre.

Je continue de chercher quand j'ai un peu de temps mais si quelqu'un a une solution, elle est bienvenue

@gcodeur

This comment has been minimized.

Copy link
Member

gcodeur commented May 11, 2017

Tiens, c'est exactement le problème qu'a eu @rezemika avec son installation et qu'on a pas réussi à résoudre. :/

Malheureusement, j'ai pas d'idée pour ça mais n'hésite pas à partager si tu trouves l'origine du problème.

@AmarOk1412

This comment has been minimized.

Copy link
Member

AmarOk1412 commented May 11, 2017

En attendant y a toujours moyen de désactiver la barre sur le coté pour tester localement @ChantyTaguan ;)

@ChantyTaguan

This comment has been minimized.

Copy link
Contributor Author

ChantyTaguan commented May 12, 2017

Bon, j'ai suivi cette doc et ça a l'air de fonctionner. A voir si garde ça juste en local ou si j'en fait une PR. A priori ça devrait rien casser chez ceux pour qui ça marchait avant mais j'ai pas vraiment étudié les implications...

@coveralls

This comment has been minimized.

Copy link

coveralls commented May 12, 2017

Coverage Status

Coverage decreased (-0.05%) to 88.685% when pulling 540e352 on ChantyTaguan:feature/tag-subscription into 96d91fd on zestedesavoir:dev.

@ChantyTaguan

This comment has been minimized.

Copy link
Contributor Author

ChantyTaguan commented May 12, 2017

Voilààààààà \o/

@AmarOk1412

This comment has been minimized.

Copy link
Member

AmarOk1412 commented May 18, 2017

QA: Ok, j'ai aussi testé le changement de tags

@artragis

This comment has been minimized.

Copy link
Member

artragis commented May 24, 2017

Bah si QA OK, on peut merger ou pas?

@artragis artragis merged commit 85fb311 into zestedesavoir:dev May 25, 2017

1 check passed

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

@gcodeur gcodeur removed the QA svp label May 30, 2017

@gustavi gustavi added this to the Version de développement milestone Jun 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.