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

[WIP] Empêche les notifications persistantes pour les pings #4197

Open
wants to merge 12 commits into
base: dev
from

Conversation

Projects
None yet
5 participants
@artragis
Contributor

artragis commented Feb 11, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) #4195, #4193
  • fix str function

QA

Déplacement vers un forum auquel le pingé n'a pas accès
    Pinger X dans un forum 'public'
    Déplacer le sujet dans un forum non-public
Créer un ping dans un forum auquel le pingé n'a pas accès
    Pinger X dans un forum non-public
Créer un ping
    editer sans enlever le ping
    le ping ne doit pas générer deux notification.

@DevHugo DevHugo added this to the v22 milestone Feb 11, 2017

@DevHugo DevHugo self-assigned this Feb 11, 2017

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Je fait la QA ce soir.

Contributor

DevHugo commented Feb 11, 2017

Je fait la QA ce soir.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Je vais faire attention, y'a pas de souci.

Contributor

DevHugo commented Feb 11, 2017

Je vais faire attention, y'a pas de souci.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor

Comme ça me faisais chier de fixer #4193 dans une PR qui allait forcément être en conflit avec celle-ci, j'ai mis le fix ici :)

Contributor

artragis commented Feb 11, 2017

Comme ça me faisais chier de fixer #4193 dans une PR qui allait forcément être en conflit avec celle-ci, j'ai mis le fix ici :)

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

No rage bro.

Contributor

DevHugo commented Feb 11, 2017

No rage bro.

Show outdated Hide outdated zds/notification/receivers.py Outdated
Show outdated Hide outdated zds/notification/receivers.py Outdated
Show outdated Hide outdated zds/notification/receivers.py Outdated
Show outdated Hide outdated zds/notification/receivers.py Outdated

@vhf vhf changed the title from Empêche les notifications persistantes pour les pings to [v22] Empêche les notifications persistantes pour les pings Feb 11, 2017

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

J'ai une erreur, mais j'arrive pas à trouver ou Cannot resolve keyword 'group' into field. Choices are: author, author_id, es_already_indexed, es_flagged, forum, forum_id, id, is_locked, is_solved, is_sticky, last_message, last_message_id, post, pubdate, publishablecontent, subtitle, tags, title, topicfollowed, topicread, update_index_date.

C'est quand je poste un topic dans la corbeille avec des ping.

Quelque part dans zds/notification/receivers.py in answer_comment_event

Contributor

DevHugo commented Feb 11, 2017

J'ai une erreur, mais j'arrive pas à trouver ou Cannot resolve keyword 'group' into field. Choices are: author, author_id, es_already_indexed, es_flagged, forum, forum_id, id, is_locked, is_solved, is_sticky, last_message, last_message_id, post, pubdate, publishablecontent, subtitle, tags, title, topicfollowed, topicread, update_index_date.

C'est quand je poste un topic dans la corbeille avec des ping.

Quelque part dans zds/notification/receivers.py in answer_comment_event

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

Show outdated Hide outdated zds/notification/receivers.py Outdated
@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Si on édite dans le forum et qu'on ajoute une personne qui n'était pas déjà ping, cette personne là, ne reçoit pas de notification.

Exemple :

  • Admin poste avec un sujet avec un ping de user et valide
  • Il édite et ajoute un ping de staff, valide l'édition
  • Y'a que user qui est ping.
Contributor

DevHugo commented Feb 11, 2017

Si on édite dans le forum et qu'on ajoute une personne qui n'était pas déjà ping, cette personne là, ne reçoit pas de notification.

Exemple :

  • Admin poste avec un sujet avec un ping de user et valide
  • Il édite et ajoute un ping de staff, valide l'édition
  • Y'a que user qui est ping.
@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

On ne peut ping personne dans les forums privé, par exemple un admin ne peut pas pinger admin2 dans le forum privé.

Contributor

DevHugo commented Feb 11, 2017

On ne peut ping personne dans les forums privé, par exemple un admin ne peut pas pinger admin2 dans le forum privé.

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Passer dans un message dans un forum privé supprime bien les notification pour ceux qui n'ont pas les droits mais aussi ceux qui ont les droits.

Contributor

DevHugo commented Feb 11, 2017

Passer dans un message dans un forum privé supprime bien les notification pour ceux qui n'ont pas les droits mais aussi ceux qui ont les droits.

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor

On ne peut ping personne dans les forums privé, par exemple un admin ne peut pas pinger admin2 dans le forum privé.

ceci devrait être corrigé et agira sûrement sur la partie "je peux pas pinger lors de l'édit"

Contributor

artragis commented Feb 11, 2017

On ne peut ping personne dans les forums privé, par exemple un admin ne peut pas pinger admin2 dans le forum privé.

ceci devrait être corrigé et agira sûrement sur la partie "je peux pas pinger lors de l'édit"

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Déplacement vers un forum auquel le pingé n'a pas accès
- Pinger X dans un forum 'public'
- Déplacer le sujet dans un forum non-public

Marche plus du coup mais le reste marche bien

Contributor

DevHugo commented Feb 11, 2017

Déplacement vers un forum auquel le pingé n'a pas accès
- Pinger X dans un forum 'public'
- Déplacer le sujet dans un forum non-public

Marche plus du coup mais le reste marche bien

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 11, 2017

Contributor
Contributor

artragis commented Feb 11, 2017

@DevHugo

This comment has been minimized.

Show comment
Hide comment
@DevHugo

DevHugo Feb 11, 2017

Contributor

Je l'ai fait vite fait, je peut-être merder sur la QA, je confirme demain.

Contributor

DevHugo commented Feb 11, 2017

Je l'ai fait vite fait, je peut-être merder sur la QA, je confirme demain.

@vhf vhf changed the title from [v22] Empêche les notifications persistantes pour les pings to Empêche les notifications persistantes pour les pings Feb 12, 2017

@vhf vhf removed this from the v22 milestone Feb 12, 2017

@vhf

This comment has been minimized.

Show comment
Hide comment
@vhf

vhf Feb 12, 2017

Member

J'ai changé la target branch vers dev au lieu de release. On activera pas les ping cette fois pour éviter de retarder une release qui devrait être assez courte et simple. Notamment parce qu'on va devoir attendre un peu de dev sur ZMarkdown. :)

Member

vhf commented Feb 12, 2017

J'ai changé la target branch vers dev au lieu de release. On activera pas les ping cette fois pour éviter de retarder une release qui devrait être assez courte et simple. Notamment parce qu'on va devoir attendre un peu de dev sur ZMarkdown. :)

@artragis

This comment has been minimized.

Show comment
Hide comment
@artragis

artragis Feb 28, 2017

Contributor

Après rebase et test, j'ai corrigé + ajouté un TU. Par contre les migrations sur les nombres de lettres dans les tutos m'empêchent de tester correctement chez moi.

Contributor

artragis commented Feb 28, 2017

Après rebase et test, j'ai corrigé + ajouté un TU. Par contre les migrations sur les nombres de lettres dans les tutos m'empêchent de tester correctement chez moi.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Feb 28, 2017

Coverage Status

Coverage increased (+0.08%) to 88.731% when pulling 5899c57 on artragis:fix_4195 into 579755b on zestedesavoir:dev.

coveralls commented Feb 28, 2017

Coverage Status

Coverage increased (+0.08%) to 88.731% when pulling 5899c57 on artragis:fix_4195 into 579755b on zestedesavoir:dev.

@vhf

This comment has been minimized.

Show comment
Hide comment

@vhf vhf changed the title from Empêche les notifications persistantes pour les pings to [WIP] Empêche les notifications persistantes pour les pings Aug 4, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@zestedesavoir zestedesavoir deleted a comment from coveralls Sep 24, 2017

@Situphen Situphen removed the P-Haute label Nov 11, 2017

@artragis artragis added the S-Zombie label Sep 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment