Projet

Général

Profil

Development #46612

annulation de réservation, ne pas appeler le callback par défaut

Ajouté par Valentin Deniaud il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
14 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Notamment pas quand on se trouve dans l'API /cancel/...


Fichiers

Révisions associées

Révision 3ac443b5 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: do not call cancel booking callback by default (#46612)

Historique

#2

Mis à jour par Valentin Deniaud il y a plus de 3 ans

#3

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
Si je comprends bien, on a cette situation maintenant :
  • 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
#4

Mis à jour par Valentin Deniaud il y a plus de 3 ans

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

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.

#5

Mis à jour par Thomas Noël il y a plus de 3 ans

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)

#6

Mis à jour par Valentin Deniaud il y a plus de 3 ans

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 ?

#7

Mis à jour par Thomas Noël il y a plus de 3 ans

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)

#8

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Solution validée à 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)
#9

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

Formats disponibles : Atom PDF