Développement #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.
Files
Associated revisions
History
Updated by Valentin Deniaud almost 3 years ago
- File 0001-agendas-allow-different-reminder-time-between-email-.patch 0001-agendas-allow-different-reminder-time-between-email-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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.
Updated by Valentin Deniaud almost 3 years ago
Updated by Thomas Noël almost 3 years ago
- Status changed from Solution proposée to 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)
Updated by Valentin Deniaud almost 3 years ago
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
Updated by Valentin Deniaud almost 3 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique almost 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
agendas: allow different reminder time between email and sms (#61368)