ajout de la possibilité de suggérer des speakers #616
Conversation
agallou
commented
May 10, 2018
agallou
commented
| <br /> | ||
| Mais cette année, nous souhaitons ouvrir une troisième possibilité : faites-nous part de vos envies, partagez avec nous vos idées de conférencier·e à contacter ! Que vous le/la connaissiez, que vous rêviez de l’écouter, ou que vous ayez déjà assisté à une de ses conférences, parlez-nous de cet·te orateur·trice : nous étudierons chaque suggestion en vue d’une invitation ou d’un appel à participation au CFP. Renseignez le formulaire ci-contre pour nous faire vos suggestions de speakers.<br /> | ||
| <br /> | ||
| Nous espérons ainsi toujours mieux vous satisfaire, tout en favorisant la diversité dans nos programme et la variété des sujets abordés.<br /> |
sources/Afup/Utils/Mail.php
Outdated
| * @param string $message | ||
| * @param string $receivers | ||
| */ | ||
| public function sendSimpleMessageViaSmtp($subject, $message, $receivers) |
There was a problem hiding this comment.
j'aime pas trop le nom de cette méthode. J'ai l'impression qu'en substance elle fait la même chose que sendSimpleMessage mais en transitant par un autre transport. Par ailleurs on avait arrêté d'exploiter petit à petit la classe Mailing car on avait de gros problèmes de delivery. Je me demande du coup si cette méthode est vraiment nécessaire: sendSimpleMessage faisant un envoi de message sans template, elle pourrait toujours être exploitée pour envoyer via mailchimp en smtp.
|
Une idée supplémentaire pour + tard: stocker aussi les données dans un spreadsheet google ? |
|
Oui, à terme cette méthode devra être supprimé une fois qu'on sera
complètement passés par le SMTP de mandrill, elle est là pour la transition.
Après vu que ce n'est pas l'objet du ticket je peux supprimer la méthode,
passer par l'API mandrill (c'est plus compliqué à tester localement, mais
devrait fonctionner). On ferais la migration complète dans une autre pr.
Ça t'irait ?
Le ven. 11 mai 2018 à 08:55, Xavier Leune <notifications@github.com> a
écrit :
… ***@***.**** commented on this pull request.
------------------------------
In sources/Afup/Utils/Mail.php
<#616 (comment)>:
> @@ -144,5 +144,18 @@ public function sendSimpleMessage($subject, $message, $receivers = null)
return $this->send('message', array(), $data, $parameters);
}
+ /**
+ * Début de migration pour être décorrolé de l'API Mandrill et passer sur un smtp gratuit.
+ *
+ * @param string $subject
+ * @param string $message
+ * @param string $receivers
+ */
+ public function sendSimpleMessageViaSmtp($subject, $message, $receivers)
j'aime pas trop le nom de cette méthode. J'ai l'impression qu'en substance
elle fait la même chose que sendSimpleMessage mais en transitant par un
autre transport. Par ailleurs on avait arrêté d'exploiter petit à petit la
classe Mailing car on avait de gros problèmes de delivery. Je me demande du
coup si cette méthode est vraiment nécessaire: sendSimpleMessage faisant un
envoi de message sans template, elle pourrait toujours être exploitée pour
envoyer via mailchimp en smtp.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#616 (review)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AATjdGv6o4DC0EwDggppRZeVxUIQxKT_ks5txUPqgaJpZM4T5rYg>
.
|
|
pour le spreadsheet, actuellement on les stocke en base (pour l'instant on a pas de de page dans le BO pour les voir, ça viendra dans un second temps, l'important était de pouvoir les saisir). Si on veux avoir les données dans un spreadsheet, on pourra rajouter un export csv dans le BO. |
|
Etant donné que cela envoi des emails, ne peut-on pas avoir un cap-chat pour se prémunir d'un bot de spam ? |
|
@cmourizard en effet, +1 pour le recaptcha invisible by google |
|
pour le préremplissage, l'issue à été crée : #621 |
