Development #39654
connecteurs SMS, messages longs ?
0%
Description
Aujourd'hui w.c.s. tronque les messages à 160 caractères mais certains fournisseurs peuvent gérer des messages longs; il y a demande pour supprimer la limitation dans w.c.s. mais ça demande peut-être alors de marquer les connecteurs SMS autorisant les messages longs et tronquer, au niveau de Passerelle, pour les autres.
Fichiers
Demandes liées
Révisions associées
base: add maximum sms length option (#39654)
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Development #39587: Ne pas tronquer les SMS ajouté
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0002-base-add-maximum-sms-length-option-39654.patch 0002-base-add-maximum-sms-length-option-39654.patch ajouté
- Fichier 0001-sms-turn-mixin-into-abstract-class-39654.patch 0001-sms-turn-mixin-into-abstract-class-39654.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le prérequis, pour ce ticket et les autres ouverts dans la foulée, c'est de pouvoir partager du code de modèle entre les backends. J'ai tenté un truc dans 0001, si quelqu'un a une meilleure manière je prends.
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0002-base-add-maximum-sms-length-option-39654.patch 0002-base-add-maximum-sms-length-option-39654.patch ajouté
- Fichier 0001-sms-turn-mixin-into-abstract-class-39654.patch 0001-sms-turn-mixin-into-abstract-class-39654.patch ajouté
Oublié les migrations.
Mis à jour par Frédéric Péters il y a environ 4 ans
Je serais même pour une hiérarchie simple et directe SMSResource(BaseResource)
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0002-base-add-maximum-sms-length-option-39654.patch 0002-base-add-maximum-sms-length-option-39654.patch ajouté
- Fichier 0001-sms-turn-mixin-into-abstract-class-39654.patch 0001-sms-turn-mixin-into-abstract-class-39654.patch ajouté
Pareil, mais je n'étais pas sûr de mon coup. Revoici.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit bb44e565a5de4540324f920d3abc1e3c8e7f3542 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 10 16:45:02 2020 +0100 base: add maximum sms length option (#39654) commit 37e7c58bd168ea280f6f350ab173699644d40829 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Mar 10 15:25:43 2020 +0100 sms: turn mixin into abstract class (#39654)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a environ 4 ans
En fait mon idée initiale c'était qu'on sache que tel fournisseur autorise les messages longs, et que tel autre ne les autorise pas, et que ça soit un flag sur le connecteur, pas une option manuelle.
Mis à jour par Valentin Deniaud il y a environ 4 ans
Sauf que dans le ticket client initial (#39342), « Or nous souhaitons utiliser l'option "SMS longs" d'Orange qui permet d'envoyer jusqu'à 459 caractères en concaténant jusqu'à 3 SMS » ce qui sans avoir vérifié me fait dire qu'Orange propose de payer plus cher pour envoyer plus long, et qu'on ne peut donc pas décider à priori.
sms: turn mixin into abstract class (#39654)