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

Courriels lors d'une réponse à un message privé aux mauvaises personnes #2905

Closed
Situphen opened this issue Jul 8, 2015 · 13 comments
Closed
Labels
Milestone

Comments

@Situphen
Copy link
Contributor

@Situphen Situphen commented Jul 8, 2015

Reporté via le forum :

Bon, on a fait quelques tests avec les compères, et l'explication qui semble se profiler, c'est que :

  • l'OP ne reçoit jamais de mail de notification ;
  • les autres reçoivent systématiquement un mail de notification, même pour leurs propres messages ;
  • le mail de notification attribue systématiquement le dernier message envoyé à l'OP.

La manière la plus économique de décrire le problème, et je pense la raison pour laquelle ça bogue, c'est que le module de ZdS chargé d'envoyer les notifs mail aux bonnes personnes considère systématiquement l'OP comme l'auteur du dernier message posté. Cela expliquerait le texte du mail, le fait que l'OP ne reçoive jamais (puisqu'il est censé l'avoir écrit) et que les autres reçoivent toujours (puisque c'est censé être l'OP qui a écrit). Je sais pas si je suis clair ?

@DevHugo

This comment has been minimized.

Copy link
Contributor

@DevHugo DevHugo commented Jul 11, 2015

C'est sur cette ligne, je pense https://github.com/zestedesavoir/zds-site/blob/dev/zds/utils/mps.py#L107.

On se demande qui a introduit le bug, hein, part en sifflotant.

@artragis

This comment has been minimized.

Copy link
Member

@artragis artragis commented Oct 13, 2015

on dirait que ça doit être fermé ça.

@artragis

This comment has been minimized.

Copy link
Member

@artragis artragis commented Oct 13, 2015

@DevHugo

This comment has been minimized.

Copy link
Contributor

@DevHugo DevHugo commented Oct 13, 2015

Je crois que j'ai vu encore des soucis en prod ... à confirmer !

@Eskimon

This comment has been minimized.

Copy link
Member

@Eskimon Eskimon commented Oct 13, 2015

Yep j'ai pas l'impression que c'est corrigé non plus d'après les PR

@Berdes

This comment has been minimized.

Copy link

@Berdes Berdes commented Oct 21, 2015

Je confirme : j'ai toujours le bug. En tout cas, je reçois un mail quand je réponds à un MP (et je ne suis pas l'OP). Je vais demander à l'OP s'il est sensé recevoir des mails de notification et s'il en reçoit réellement.

@DevHugo

This comment has been minimized.

Copy link
Contributor

@DevHugo DevHugo commented Oct 21, 2015

Le nom est bon dans le mail ?
Le 21 oct. 2015 11:19 PM, "Berdes" notifications@github.com a écrit :

Je confirme : j'ai toujours le bug. En tout cas, je reçois un mail quand
je répond à un MP (et je ne suis pas l'OP). Je vais demander à l'OP s'il
est sensé recevoir des mails de notification et s'il en reçoit réellement.


Reply to this email directly or view it on GitHub
#2905 (comment)
.

@Berdes

This comment has been minimized.

Copy link

@Berdes Berdes commented Oct 21, 2015

Dans le mail que je reçois (et comme dans le bug original), c'est le nom de l'OP qui apparaît comme expéditeur de ma réponse.

@Berdes

This comment has been minimized.

Copy link

@Berdes Berdes commented Oct 22, 2015

L'OP viens de me dire qu'il n'est pas sensé recevoir de notification et qu'il n'en reçoit pas. Donc ça colle a priori au bug de base.

@DevHugo

This comment has been minimized.

Copy link
Contributor

@DevHugo DevHugo commented Oct 31, 2015

Ça le fait toujours ?

@firm1

This comment has been minimized.

Copy link
Contributor

@firm1 firm1 commented Oct 31, 2015

Toujours chez moi

Le sam. 31 oct. 2015 15:17, Hugo Courtecuisse notifications@github.com a
écrit :

Ça le fait toujours ?


Reply to this email directly or view it on GitHub
#2905 (comment)
.

@GerardPaligot

This comment has been minimized.

Copy link
Member

@GerardPaligot GerardPaligot commented Feb 13, 2016

Cette issue devrait être corrigée par la ZEP-24, même déjà dans ma PR. Je la mentionne dans ma PR.

@gustavi

This comment has been minimized.

Copy link
Member

@gustavi gustavi commented Mar 5, 2016

Mergé dans ZEP-24.

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