Development #49034
inclure la description de l'événement dans le mail de rappel
0%
Description
Ça permettrait de répondre très facilement au besoin de l'envoi d'un rappel avec une description à jour.
(plutôt qu'avoir l'envoi depuis w.c.s. où la description a été enregistrée/figée au moment de la réservation).
Files
Related issues
Associated revisions
History
Updated by Emmanuel Cazenave almost 3 years ago
- Related to Development #48877: API : renvoyer la description de l'évènement sur le endpoint slot_status added
Updated by Frédéric Péters almost 3 years ago
(description + autres infos sur l'événement genre l'URL)
Updated by Valentin Deniaud almost 3 years ago
- File 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Je pense qu'il faudrait faire mieux que les « {{ event_label }} », « {{ event_description }} » et autres « {{ user_display_label }} » qui s'affichent tels quels dans la vue de prévisualisation du mail, parce qu'en plus d'être en anglais ça commence à faire lourd, non ?
Updated by Frédéric Péters almost 3 years ago
Je pense qu'il faudrait faire mieux que les « {{ event_label }} », « {{ event_description }} » et autres « {{ user_display_label }} » qui s'affichent tels quels dans la vue de prévisualisation du mail, parce qu'en plus d'être en anglais ça commence à faire lourd, non ?
En gros avoir des vrais textes d'exemple ?
Tu peux juste tirer les variables d'un événement pioché au hasard sur la page ?
Updated by Valentin Deniaud almost 3 years ago
- File 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch added
Frédéric Péters a écrit :
Tu peux juste tirer les variables d'un événement pioché au hasard sur la page ?
Ouais mais est-ce que l'évènement aura bien une description, une url et un prix, sinon on ne saura jamais que le mail peut les contenir, et puis vraiment au hasard genre ça change à chaque prévisualisation, ou alors critère arbitraire genre le dernier créé, ou le premier encore ouvert, et puis d'abord pourquoi je peux pas choisir l'évènement ? Bref moyennement envie d'aller par là (enfin, ajouter un select pour permettre de choisir un vrai évènement oui, mais pas dans ce ticket).
J'améliore tout de même les textes d'exemple en virant les accolades et ça ira bien comme ça (je garde volontairement {{ user_booking_label }}
qui je trouve a un sens car pas de champ de formulaire correspondant).
Updated by Frédéric Péters almost 3 years ago
Tu pourrais aussi générer du texte (from django.utils import lorem_ipsum) + ajouter derrière quelques mots sur celui-ci, dans l'esprit de la capture attachée ?
Updated by Frédéric Péters almost 3 years ago
et aussi, sur la description, je serais pour faire |linebreaks.
Updated by Valentin Deniaud almost 3 years ago
Frédéric Péters a écrit :
Tu pourrais aussi générer du texte (from django.utils import lorem_ipsum) + ajouter derrière quelques mots sur celui-ci, dans l'esprit de la capture attachée ?
Dac, et quitte à préciser les choses, tu réserverais quel sort à l'URL de l'évènement ?
Updated by Frédéric Péters almost 3 years ago
En text/html sous forme de lien, <a href="(l'url)">Plus d’informations</a>; dans la prévisu avoir <a href="#">Plus d’informations</a> <small>(lien vers l’adresse de l’événement, si présente)</small>.
En text/plain, Plus d’informations : l'url.
Updated by Frédéric Péters almost 3 years ago
Additionnal → Additional (un seul n).
<p><a href="{{ booking.event.url }}">{% trans "More information" %}</a>
Ça peut produire deux boutons l'un au-dessus de l'autre mais je serais quand même pour utiliser {% include "emails/button-link.html" ... %} ici.
Updated by Valentin Deniaud almost 3 years ago
- File Screenshot_2020-12-14 Agendas - auie(1).png Screenshot_2020-12-14 Agendas - auie(1).png added
- File 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch added
OK mais l'enchaînement des deux boutons ne rend pas forcément top.
Updated by Frédéric Péters almost 3 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud almost 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit abdd06ad6b71c5a5cb5ccd72c68b5994ebbfeee5 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Dec 8 17:24:03 2020 +0100 agendas: include event description in reminders (#49034)
Updated by Frédéric Péters almost 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
agendas: include event description in reminders (#49034)