Development #61368
Rappels, permettre une temporalité différente entre mail et SMS
0%
Description
Actuellement on a un select 1/2/3 jours commun aux deux champs, il s'agit de le dédoubler.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-agendas-allow-different-reminder-time-between-email-.patch 0001-agendas-allow-different-reminder-time-between-email-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Actuellement : il y a un délai commun pour email et sms, puis des cases à cocher pour dire si on veut envoyer email, sms ou les deux.
Après ce patch : il y a un délai pour email, un pour sms, et donc plus besoin des cases à cocher (l'interface s'en trouve plutôt améliorée).
Rien de très spécial niveau code, duplication du champ délai, migration des données, suppression des champs devenus obsolètes.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Une fois n'est pas coutume, en attendant mypy, je verrais bien un
assert msg_type in ('sms', 'email'), 'Bad msg_type'
au début de notify et send_reminder, histoire que ces fonctions ne soient pas un jour utilisé autrement.
Mais pas d'obligation.
Je note que le code gagne aussi en clarté, pas seulement l'UI.
Je ne dis rien sur les f-string, on est 2022, pyupgrade, toussa, #okboomer.
Au passage, sur les traductions, quand même, on pourrait en profiter pour passer de
Dans le but d’empêcher les usagers de recevoir un rappel peu de temps après ...
à
Pour éviter un rappel peu de temps après ... (et adapter la suite de la phrase)
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Thomas Noël a écrit :
histoire que ces fonctions ne soient pas un jour utilisé autrement.
Mais pas d'obligation.
Je profite de l'absence d'obligation, le problème potentiel me paraît trop improbable pour mériter une protection.
Au passage, sur les traductions, quand même, on pourrait en profiter pour passer de
Cool je vais appliquer ça
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9b3580973e546ff04309ef27383908665b1585a1 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Feb 2 17:45:15 2022 +0100 agendas: allow different reminder time between email and sms (#61368)
Mis à jour par Transition automatique il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agendas: allow different reminder time between email and sms (#61368)