Produits Entr'ouvert » Corbo

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 ?

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

Ack.