Skip to content

Netsize Support + SMS service refactos

Vincent Agnano requested to merge feature/sms-netsize into master

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

Merge request reports