Development #12665
diffusion par SMS
0%
Description
Pour des alertes/urgences, certaines annonces aux citoyens devraient pouvoir être envoyées par SMS.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 7 ans
- Lié à Development #12918: séparer les annonces par mail des alertes SMS ajouté
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Lié à Development #19846: faire evoluer l'API pour la prise en compte des abonnements par SMS ajouté
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier 0001-add-sms-delivery-12665.patch 0001-add-sms-delivery-12665.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Serghei Mihai
- Patch proposed changé de Non à Oui
Avec l'url vers la passerelle de messages et l'expéditeur à préciser dans le settings.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Revoir send pour le découper en send_email, send_sms, etc.. en prévision de MMS, WebNotification, etc.. en bouclant sur channel_choices:
for channel, name in channel_choices: getattr(self, 'send_' + channel)()
chaque send_* se débrouille pour filtrer ce qui lui plaît (on peut vraisemblablement ajouter une méthode helper sur la même classe du genre def filter_destinations(self, prefix)
.
Si j'étais Corbo je me plaindrai si il n'y a pas de SMS_GATEWAY_URL mais qu'on reçoit des demande d'envoi de SMS (un simple log.error() suffira à ce qu'on reçoit un mail pour nous faire penser à corriger ça) (et tu pourras modifier test_send_sms_with_no_gateway_defined
pour vérifier que dans caplog.records t'as bien ton erreur qui remonte)).
Je reviendrai sur l'idée de ne mettre que l'email dans le token de désinscription, faut y mettre la chaîne complète mailto:*
et ainsi ça marchera aussi pour le mail, sachant que les URLs sont désormais cliquables dans tous les SMS, on peut aussi y mettre une URL de désinscription (idéalement on utiliserait un raccourcisseur d'URL mais bon), et donc ça supprime la modification dans corbo/views.py qui devient inutile.
Je vois dans les tests qu'on ne stockait que l'email dans le Subscription.identifier, c'est les tests qui déconnaient ou c'est comme ça quelque part en production ? Faudrait pas une migration de donnée dans ce cas ?
L'ajout de nltk dans requirements.txt c'est pour quoi faire ?
Mis à jour par Serghei Mihai il y a plus de 6 ans
Benjamin Dauvergne a écrit :
[...]
chaque send_* se débrouille pour filtrer ce qui lui plaît (on peut vraisemblablement ajouter une méthode helper sur la même classe du genre
def filter_destinations(self, prefix)
.
Ok.
Si j'étais Corbo je me plaindrai si il n'y a pas de SMS_GATEWAY_URL mais qu'on reçoit des demande d'envoi de SMS (un simple log.error() suffira à ce qu'on reçoit un mail pour nous faire penser à corriger ça) (et tu pourras modifier
test_send_sms_with_no_gateway_defined
pour vérifier que dans caplog.records t'as bien ton erreur qui remonte)).
D'accord.
Je vois dans les tests qu'on ne stockait que l'email dans le Subscription.identifier, c'est les tests qui déconnaient ou c'est comme ça quelque part en production ? Faudrait pas une migration de donnée dans ce cas ?
C'est parce que l'api d'abonnement préfixe automatiquement le mail par "mailto". C'était l'idée initiale d'implementer l'envoi des mails d'abord. Donc dans Subscription
tous les identifiants contiennent "mailto:<email>".
L'ajout de nltk dans requirements.txt c'est pour quoi faire ?
Une scorie suite à mes tentatives de convertir le contenu HTML en pur texte. Mais j'ai découvert que lxml savait le faire.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Serghei Mihai a écrit :
C'est parce que l'api d'abonnement préfixe automatiquement le mail par "mailto". C'était l'idée initiale d'implementer l'envoi des mails d'abord. Donc dans
Subscription
tous les identifiants contiennent "mailto:<email>".
Je comprends bien, mais je ne vois pas comment dans les tests on se retrouvait avec ça:
identifier='%s@example.net' % uuid
sans le mailto;.
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 9f832011653bb88f8fa4a08c412469addbce2327 (origin/master) Author: Serghei Mihai <smihai@entrouvert.com> Date: Mon Aug 7 16:50:47 2017 +0200 add sms delivery (#12665)
add sms delivery (#12665)