Développement #46444
Pouvoir paramétrer les sms
0%
Description
Il y a chrono qui a envie de savoir en envoyer (#45293) et ça pourrait aussi être utilisé par w.c.s.
Ça serait fait en posant des settings, je vois pour l'instant juste l'URL à appeler et le from.
Files
Related issues
Associated revisions
History
Updated by Thomas Noël over 4 years ago
- Expéditeur (nom ou numéro)
- URL
Updated by Valentin Deniaud about 4 years ago
- File 0001-misc-add-sms-configuration-46444.patch 0001-misc-add-sms-configuration-46444.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Voilà, tout bateau.
Et donc si j'ai bien compris, ces nouvelles variables doivent être récupérées dans les autres briques via genre settings.TEMPLATE_VARS['sms_url'] ?
Updated by Frédéric Péters about 4 years ago
Et donc si j'ai bien compris, ces nouvelles variables doivent être récupérées dans les autres briques via genre settings.TEMPLATE_VARS['sms_url'] ?
Oui mais non, plutôt il faut modifier le loader TemplateVars, dans hobo/multitenant/settings_loaders.py, pour aller poser ça sous les noms définis dans chrono, SMS_URL & SMS_FROM, comme c'est fait pour DEFAULT_FROM_EMAIL.
En rapport, #46712.
Updated by Valentin Deniaud about 4 years ago
Frédéric Péters a écrit :
plutôt il faut modifier le loader TemplateVars, dans hobo/multitenant/settings_loaders.py
C'était en effet le sens de ma question.
Updated by Thomas Noël about 4 years ago
Sender name or phone number, which length cannot exceed 11 characters.
La contrainte est plus lourde : uniquement des lettres ASCII (sans accent), des chiffres et des espaces. Aucun caractère spécial ("certains téléphones n’affichent pas les caractères de ponctuation dans les expéditeurs.").
Ca serait bien de mettre un validateur correspondant genre « [A-Za-z0-9 ]{0,11} »
Sur « sms_url = forms.URLField » je me tâte un peu parce qu'on aimerait poser du "{{passerelle_url}}" mais je ne pense pas que Chrono gère encore ça... allez, laissons URLField.
Le reste m'a l'air tout bon.
Updated by Frédéric Péters about 4 years ago
Sur « sms_url = forms.URLField » je me tâte un peu parce qu'on aimerait poser du "{{passerelle_url}}" mais je ne pense pas que Chrono gère encore ça... allez, laissons URLField.
Mais Hobo pourrait faire le taf de conversion dans le settings loader.
Updated by Valentin Deniaud about 4 years ago
J'ai mis la regex, pour {{ passerelle_url }} ça sera dans un autre ticket, parce que ni chrono ni hobo ne savent faire et de ce que je lis dans combo il y a quand même un peu de code à rapatrier pour faire ça bien.
Updated by Thomas Noël about 4 years ago
Valentin Deniaud a écrit :
J'ai mis la regex, pour {{ passerelle_url }} ça sera dans un autre ticket, parce que ni chrono ni hobo ne savent faire et de ce que je lis dans combo il y a quand même un peu de code à rapatrier pour faire ça bien.
Mmmh.. en fait hobo connait les templates_vars, donc faire un coup de « tenant_settings.SMS_URL = template(sms_url).render(context=templatevars) » ça pourrait s'imaginer.
Mais si ça demande de bouger bcp de code, on peut déporter à un autre ticket.
Updated by Frédéric Péters about 4 years ago
- Status changed from Solution proposée to Solution validée
Je vais déjà pousser ça et je créerai un ticket pour la suite.
Updated by Frédéric Péters about 4 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 538963b8ba4b12f259b3fa45260981da0493cfa2
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Sep 16 16:48:49 2020 +0200
misc: add sms configuration (#46444)
Updated by Frédéric Péters about 4 years ago
- Related to Développement #46779: hobo deploy, interpréter les variables du paramétrage SMS added
Updated by Frédéric Péters about 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: add sms configuration (#46444)