Développement #46612
annulation de réservation, ne pas appeler le callback par défaut
0%
Description
Notamment pas quand on se trouve dans l'API /cancel/...
Files
Associated revisions
History
Updated by Valentin Deniaud about 4 years ago
- File 0001-agendas-do-not-call-cancel-booking-callback-by-defau.patch 0001-agendas-do-not-call-cancel-booking-callback-by-defau.patch added
- Tracker changed from Bug to Développement
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Thomas Noël about 4 years ago
- Status changed from Solution proposée to Solution validée
- sur chrono, quand on clique pour annuler un rendez-vous, par défaut ça déclenche sur wcs (mais on peut cocher "ne pas")
- si annulation via une commande en ligne, ça déclenche sur wcs
- si annulation via l'API, ça ne déclenche pas sur wcs
Updated by Valentin Deniaud about 4 years ago
- l'usager clique sur annuler
- le wf passe dans le statut annulé
- l'appel webservice d'annulation échoue
Et donc demande dans le statut annulé mais pas d'annulation dans chrono, pas cool... Ça mérite peut-être un hotfix ?
Après je doute que des wf de prod aient déjà été modifiés pour inclure ces mécanismes, ie envoyer cancel_callback_url à la réservation.
Updated by Thomas Noël about 4 years ago
Valentin Deniaud a écrit :
Yep. Et là le bug que ça déclenchait c'est quand même :
- l'usager clique sur annuler
- le wf passe dans le statut annulé
- l'appel webservice d'annulation échoue
Mais heu donc oui, heu, ton patch corrige ça non ? (l'appel ws ne va pas échouer car il ne fera pas de callback wcs)
Updated by Valentin Deniaud about 4 years ago
Thomas Noël a écrit :
Mais heu donc oui, heu, ton patch corrige ça non ? (l'appel ws ne va pas échouer car il ne fera pas de callback wcs)
Oui, j'explicitais juste le bug initial pour contextualiser l'interrogation, hotfix ou pas ?
Updated by Thomas Noël about 4 years ago
Valentin Deniaud a écrit :
Thomas Noël a écrit :
Mais heu donc oui, heu, ton patch corrige ça non ? (l'appel ws ne va pas échouer car il ne fera pas de callback wcs)
Oui, j'explicitais juste le bug initial pour contextualiser l'interrogation, hotfix ou pas ?
Ah oui, oui, on va hotfixer (mais poussé déjà sur main)
Updated by Valentin Deniaud about 4 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 3ac443b5c32ba6c4ffffa141ad128c7213896ec6
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon Sep 14 13:58:41 2020 +0200
agendas: do not call cancel booking callback by default (#46612)Updated by Frédéric Péters about 4 years ago
- Status changed from Résolu (à déployer) to Solution déployée
agendas: do not call cancel booking callback by default (#46612)