Ajout de l'adresse postale dans le formulaire d'adhésion à l'asso #652
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Pour le coup, j'aurais bien aimé un test en plus pour vérifier que le code tourne bien. Là j'ai du le lire de long en large :) Donc ça me semble ok, je merge. |
firm1
pushed a commit
that referenced
this pull request
Jun 1, 2014
Ajout de l'adresse postale dans le formulaire d'adhésion à l'asso
|
Et pour ma culture, tu aurais vu ca comment comme test ? Une verification du contenu de l'email envoyé en fonction des champs d'entrée ? |
Il aurait fallu une simple vérification qu'un mail est parti dans la boite d'envoi pour de vrai. Pour tester ça en général, il suffit de modifier le backend des emails comme ici et ensuite vérifier le nombre de mails présents à la fin du traitement dans la boite d'envoi comme ici |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ajout de l'adresse postale dans le formulaire d'adhésion à l'asso
Le formulaire a été enrichie de nouveaux champs. Il est maintenant plus long et mériterait de tenir sur deux colonnes pour certains champs.
Le reply_to a également été viré (voir la discussion de l'issue)