Development #45742
annulation de réservation, faire du lien vers la demande un lien
0%
Description
Occupé à traduire,
#: manager/templates/chrono/manager_confirm_booking_cancellation.html:13
"This booking has no callback url configured, cancellation must be handled "
"from corresponding form: %(backoffice_url)s."
Ça serait plus agréable d’avoir un vrai lien, genre,
<p>This booking has no callback url configured, cancellation must be handled from corresponding form.</p> <a href="%(backoffice_url)s">Open form</a>
Fichiers
Révisions associées
templates: improve booking cancellation confirmation (#45742)
Historique
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0002-templates-improve-booking-cancellation-confirmation-.patch 0002-templates-improve-booking-cancellation-confirmation-.patch ajouté
- Fichier 0001-agendas-add-str-method-to-Booking-45742.patch 0001-agendas-add-str-method-to-Booking-45742.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Et la popup était moche de manière générale, j'en profite pour l'améliorer un coup.
Notamment, c'est facile de cliquer sur la croix d'à côté et d'annuler la mauvaise réservation, du coup je me dis qu'il faudrait l'afficher dans le message de confirmation. D'où 0001, où j'ai eu besoin d'écrire une méthode qui me donne la string que je veux afficher (j'aurais bien juste écrit str, mais ça oblige à vérifier le type de booking.event.agenda et ça fait des queries en plus).
Mis à jour par Frédéric Péters il y a plus de 3 ans
- <li><a {% if booking.backoffice_url %}href="{{ booking.backoffice_url }}"{% endif %}>{% if booking.user_name %}{{ booking.user_name }}{% else %}{% trans "Unknown" %}{% endif %},
- {{ booking.creation_datetime|date:"DATETIME_FORMAT" }}</a>
+ <li><a {% if booking.backoffice_url %}href="{{ booking.backoffice_url }}"{% endif %}>{{ booking }}</a>
Ça doit être {{ booking.events_display }} ici, ou pas si ça varie selon le type d'agenda, en tout cas pas juste {{ booking }}.
Et ça doit peut-être amener à ce que tu écrivais :
(j'aurais bien juste écrit str, mais ça oblige à vérifier le type de booking.event.agenda et ça fait des queries en plus).
Donc pou réviter ces appels, ça amène sans doute à passer un paramètre aux
{% include 'chrono/manager_agenda_event_fragment.html' %}
pour que le fragment sache quelle méthode appeler.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0002-templates-improve-booking-cancellation-confirmation-.patch 0002-templates-improve-booking-cancellation-confirmation-.patch ajouté
- Fichier 0001-agendas-add-str-method-to-Booking-45742.patch 0001-agendas-add-str-method-to-Booking-45742.patch ajouté
Ça doit être {{ booking.events_display }} ici
Yep, bien vu.
ou pas si ça varie selon le type d'agenda
Non, c'était bien juste une typo, et donc pas besoin de la suite.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8e56a48b4f18b27363f77840e17ff1305bef99f9
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Aug 5 10:36:52 2020 +0200
templates: improve booking cancellation confirmation (#45742)
commit e34ffe5ca0d0cd8a1c1fc007e52bd012dda35376
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Aug 5 10:28:16 2020 +0200
agendas: add str method to Booking (#45742)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
agendas: add str method to Booking (#45742)