Project

General

Profile

Bug #50116

La réservation multiple d'évènements ne fait pas bon ménage avec l'annulation en BO

Added by Valentin Deniaud almost 4 years ago. Updated almost 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Lauréline Guérin
Category:
-
Target version:
-
Start date:
13 January 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Lors de la réservation de plusieurs évènements, l'api retourne une seule URL d'annulation, qui correspond à la réservation primaire qui annule les autres en cascade.

Problème, en backoffice on peut aller annuler des réservations secondaires.

Par exemple, si je réserve A, B, C, et que A est la réservation primaire, je peux aller annuler B, il restera A et C, mais ensuite j'annule A et C disparaît, ça ne va pas.

Files

0001-manager-it-is-not-possible-to-cancel-secondary-booki.patch (4.03 KB) 0001-manager-it-is-not-possible-to-cancel-secondary-booki.patch Lauréline Guérin, 21 January 2021 10:48 AM
Capture d’écran de 2021-01-21 10-44-11.png (34.8 KB) Capture d’écran de 2021-01-21 10-44-11.png Lauréline Guérin, 21 January 2021 10:49 AM
0001-manager-it-is-not-possible-to-cancel-secondary-booki.patch (4.52 KB) 0001-manager-it-is-not-possible-to-cancel-secondary-booki.patch Lauréline Guérin, 21 January 2021 04:15 PM
Capture d’écran de 2021-01-21 16-13-37.png (35.8 KB) Capture d’écran de 2021-01-21 16-13-37.png Lauréline Guérin, 21 January 2021 04:15 PM

Associated revisions

Revision 93b8aa2c (diff)
Added by Lauréline Guérin almost 4 years ago

manager: it is not possible to cancel secondary booking (#50116)

History

#1

Updated by Lauréline Guérin almost 4 years ago

Dans l'api on refuse les actions sur les réservations secondaires.
Proposition: ne pas permettre d'annuler une réservation secondaire via l'UI

#2

Updated by Lauréline Guérin almost 4 years ago

  • Assignee set to Lauréline Guérin
#3

Updated by Lauréline Guérin almost 4 years ago

#5

Updated by Valentin Deniaud almost 4 years ago

Lauréline Guerin a écrit :

bon visuellement c'est pas terrible

Plutôt que de cacher le bouton, ajouter class="disabled" + un attribut title explicatif ?

#7

Updated by Valentin Deniaud almost 4 years ago

  • Status changed from Solution proposée to Solution validée

Nickel, il y a juste un {% if booking.primary_booking %}disabled{% endif %} en trop à retirer.

#8

Updated by Lauréline Guérin almost 4 years ago

pff, fatigue, je corrige

#9

Updated by Lauréline Guérin almost 4 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 93b8aa2c02f7f339b8a8e250bea84831c22ade11
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Jan 21 10:47:59 2021 +0100

    manager: it is not possible to cancel secondary booking (#50116)
#10

Updated by Frédéric Péters almost 4 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF