Netsize Support + SMS service refactos
Created by: adipasquale
1er commit : refacto des services SMS
Le but initial était d'isoler plus clairement la partie envoi de SMS, qui va varier selon le provider, de la partie génération du SMS.
Dans la foulée, je suis allé trop loin, mais je n'ai pas pu m'empêcher de refacto la partie génération du SMS qui ne me plaisait vraiment pas avec un send(@type)
assez bizarre. J'ai séparé chaque événement dans une classe à part comme TransactionalSms::RdvCancelled
, et elles partagent du comportement via un concern TransactionalSms::BaseConcern
qui contient par exemple le footer commun, ou la fonction pour éviter les caractères spéciaux.
2e commit : support de netsize
J'ai laissé le support de SendInBlue pour l'instant, car on va vouloir consommer le crédit restant. J'ai mis le switch sur une variable d'env.
rien de spécial pour le support Netsize; je me suis appuyé sur le script préliminaire de Thomas dans l'issue, merci ! J'ai juste du rajouter le charset utf-8 sinon l'encodage était cassé, ce n'était pas documenté. Les tests ont bien fonctionné, rien à signaler
3e commit: amélioration des tags
Les tags des sms permettront de faire des stats. J'ai rajouté un préfixe devant l'id de l'organisation : 883 -> org-883
, et rajouté un tag département : dpt-77
.
Tests sur la review app
Il faut toggler FORCE_SMS_PROVIDER=send_in_blue
sur la review app pour passer de SIB a netsize