-
Notifications
You must be signed in to change notification settings - Fork 161
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
MP à soi-même #1298
Comments
On ne peut plus s'envoyer de MP a soi même - #1298
C'est mergé, je ferme ! |
Il faudrait réouvrir de ce côté ! Le bug est toujours présent. |
Oui et non. En gros, il manque un test pour dire que si, après vérification des pseudos, la liste des destinataire est vide, ça passe pas. |
Du coup, c'est vraiment un bug : on peut actuellement envoyer un MP sans participants. |
Je prend. Je vois comment régler ça. J'en profiterais pour nous rajouter de |
Vas sur IRC alors, @Florianboux est deja dessus |
Je n'ai encore rien fait, étant donné que je ne suis pas arriver à reproduire le bug en local. Donc si il sait comment régler ça, rapidement, je laisse la main ! ;-) |
Bon. À fortiori, la réouverture n'as pas lieu d'être. En effet, la dernière PR a avoir touché à ça (#1378) n'as pas été incluse dans la dernière MEP. Comme on arrive pas à reproduire le bug sur master (mais bien sur prod et dev), j'en déduis qu'il faut "simplement" attendre que ça passe en prod' :) |
Je propose qu'on attende la prochaine MEP et qu'on ferme a ce moment la si tout va bien... |
Le bug n'est corrigé qu'à moitié : je suis bloqué si j'essaie d'envoyer un MP à SpaceFox, mais pas à spacefox. C'est complexe à tester parce que cet imb****e de SQLite n'est pas sensible à la casse, contrairement à MySQL... Le bug étant un bug v1.x mergé par convenance, il ne bloque pas la MEP de la v1.0 (sauf si bien sûr quelqu'un trouve un fix rapide). |
Y'à un fix fait par @poulp et qui agit non plus sur le formulaire mais directement sur la fonction d'envoi pour bloquer les trucs a ce moment là. Mais comme l'as dis spacefox, c'est chaud à tester ... |
Cette PR a réglé le souci non ? #1431 (dans ce cas : v1.1 et a fermer :) ) |
Ok pour fermeture ? |
J'attendrais vraiment la préprod pour être certain (faudrait un tag "préprod" pour ce genre de cas, genre on ferme mais on place le tag pour y faire attention). |
Si on fait ça avec toutes les issues, on a pas fini... si on voit que le soucis est encore présent ou rouvrira, mais si on s'amuser à laisser ouvertes toutes les issues tant qu'elles ne sont pas en prod, ça va être infernal. |
Bon... je viens de tester en preprod, si j'ecris a EsKimOn je peux bien m'ecrire a moi meme... |
C'est normal, il y avait une opération documentée je ne sais où qui donc n'a pas été faite... |
Ok... Faudra prevenir quand ca sera ok alors, histoire de pas oublier de retester :) |
J'en parle ici 2014-09-25 10:07 GMT+02:00 Eskimon notifications@github.com:
|
Rien n'empêche de s'envoyer un MP à soit-même (puisqu'on peut se sélectionner soit-même dans la liste des participants), mais aussi et surtout, rien n'empêche d'être le seul dans la conversation.
The text was updated successfully, but these errors were encountered: