Projet

Général

Profil

Development #12665

diffusion par SMS

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
20 juillet 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour des alertes/urgences, certaines annonces aux citoyens devraient pouvoir être envoyées par SMS.


Fichiers

0001-add-sms-delivery-12665.patch (12,9 ko) 0001-add-sms-delivery-12665.patch Serghei Mihai, 02 novembre 2017 01:28
0001-add-sms-delivery-12665.patch (12,6 ko) 0001-add-sms-delivery-12665.patch Serghei Mihai, 10 novembre 2017 11:26

Demandes liées

Lié à Corbo - Development #12918: séparer les annonces par mail des alertes SMSFermé24 août 2016

Actions
Lié à Corbo - Development #19846: faire evoluer l'API pour la prise en compte des abonnements par SMSFermé02 novembre 2017

Actions

Révisions associées

Révision 9f832011 (diff)
Ajouté par Serghei Mihai il y a plus de 6 ans

add sms delivery (#12665)

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 7 ans

#2

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é
#3

Mis à jour par Serghei Mihai il y a plus de 6 ans

Avec l'url vers la passerelle de messages et l'expéditeur à préciser dans le settings.

#4

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 ?

#5

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.

#6

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;.

#7

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

Ack.

#8

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)
#9

Mis à jour par Serghei Mihai il y a plus de 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF