Development #46712
ne pas utiliser hasattr pour déterminer s'il y a gestion de SMS
0%
Description
chrono/manager/forms.py: if not hasattr(settings, 'SMS_URL'):
Plutôt taper un SMS_URL = None ou "" dans les settings, et pareil pour SMS_FROM. Ça permettra à la variable d'exister, posée par Hobo, même vide.
Du coup modifier aussi chrono/agendas/management/commands/send_booking_reminders.py pour juste faire settings.SMS_URL plutôt qu'utiliser getattr().
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0002-misc-access-SMS-settings-directly-46712.patch 0002-misc-access-SMS-settings-directly-46712.patch ajouté
- Fichier 0001-misc-rename-SMS_FROM-setting-to-SMS_SENDER-46712.patch 0001-misc-rename-SMS_FROM-setting-to-SMS_SENDER-46712.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Yep, par contre j'aime plus le nom SMS_FROM, hors de l'API passerelle on comprend pas trop ce que c'est, bref 0001 tant qu'il est temps de changer ça.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Serghei Mihai a écrit :
SMS_EXPEDITOR?
Bof, ça sent le franglais ce mot, genre https://www.wordreference.com/fren/exp%C3%A9diteur (expéditeur se traduit en sender et réciproquement).
Et puis sender ça sous-entend « sender ID », le vocabulaire consacré quand on parle de SMS il me semble.
Mis à jour par Thomas Noël il y a plus de 3 ans
par parallélisme avec DEFAULT_FROM_EMAIL : DEFAULT_FROM_SMS ? (dans l'idée que ça peut aussi être modifié).
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Je m'inscrivais plutôt contre une analogie entre mails et sms, notamment que le from des mails étant une adresse mail on s'attend à ce que le from des sms soit un numéro de téléphone, alors que non.
Mis à jour par Thomas Noël il y a plus de 3 ans
Valentin Deniaud a écrit :
Je m'inscrivais plutôt contre une analogie entre mails et sms, notamment que le from des mails étant une adresse mail on s'attend à ce que le from des sms soit un numéro de téléphone, alors que non.
Bienheureux les gens qui n'ont connu que les mails rfc822 :-)
Mais donc oui, SMS_SENDER si tu veux, par contre ne faut-il pas faire un seul patch ?
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4a106ef9c7965e2e042f1a0e2784ca1b28b465ec
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Sep 17 12:13:15 2020 +0200
misc: access SMS settings directly (#46712)Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: access SMS settings directly (#46712)