Development #70667
smsfactor: la condition nostop est inversée
Début:
25 octobre 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
On a :
stop = nostop is None
donc si nostop pas dans la querystring on ajoute stop, si nostop, stop = False, mais dans smsfactor :
'pushtype': 'alert' if kwargs.get('stop', False) else 'marketing',
alert et marketing sont inversés ici.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-smsfactor-set-pushtype-alert-if-nostop-is-present-70.patch 0001-smsfactor-set-pushtype-alert-if-nostop-is-present-70.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Je constate qu'il n'y aucun test sur le comportement nostop, je vais regarder pour ça.
Mis à jour par A. Berriot il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Effectivement, j'ai du m'embrouiller les pinceaux lors de l'implem, merci pour le rétablissement du comportement attendu :)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit db88c7d6403548602485aaef2fc6042c0174094e Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Oct 25 14:42:19 2022 +0200 smsfactor: set pushtype=alert if nostop is present (#70667)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
smsfactor: set pushtype=alert if nostop is present (#70667)