search

afup / aperophp

Nouvelle version du site aperophp.net
aperophp.net
21 stars 16 forks source link

Ne pas envoyer de notification lorsque l'utilisateur choisit "Ne sera pas là" #135

Closed jubianchi closed 9 years ago

jubianchi commented 10 years ago

Je me suis inscrit aujourd'hui à cet apéro en choisissant "Ne sera pas là" : http://www.aperophp.net/315/view.html

j'ai reçu un mail de notification avec un rappel pour l'évènement à ajouter au calendrier.

Je ne suis pas certain que cela soit utile sachant que la personne ne viendra pas.

C-Duv commented 10 years ago

Quasiment 12 mois après mais j'avais envie de regarder à quoi l'application ressemblait à l'intérieur...

mikaelrandy commented 9 years ago

;)

Tu vas faire une PR ? Il manque encore les tests associés

C-Duv commented 9 years ago

Oui je compte bien faire une PR :) Je cherchais s'il y avait moyen de transformer une issue en PR ou s'il fallait que je fasse une PR en indiquant l'issue concernée.

Mais en effet il manque les tests, peut être ce week-end ;)

mikaelrandy commented 9 years ago

Il faut ouvrir une issue, et dans le corps , tu mets un texte comme "fix #" ou "close #" et le merge de la PR ferme automatiquement l'issue concernée

mikaelrandy commented 9 years ago

Closed by #120

jubianchi commented 9 years ago

:+1:

C-Duv commented 9 years ago

\o/ même pas besoin de faire de PR ni de test finalement :+1: