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).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a plus de 3 ans
- Lié à Development #48877: API : renvoyer la description de l'évènement sur le endpoint slot_status ajouté
Mis à jour par Frédéric Péters il y a plus de 3 ans
(description + autres infos sur l'événement genre l'URL)
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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 ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
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 ?
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch ajouté
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).
Mis à jour par Frédéric Péters il y a plus de 3 ans
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 ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
et aussi, sur la description, je serais pour faire |linebreaks.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
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 ?
Mis à jour par Frédéric Péters il y a plus de 3 ans
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.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch ajouté
Voilà.
Mis à jour par Frédéric Péters il y a plus de 3 ans
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.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier Screenshot_2020-12-14 Agendas - auie(1).png Screenshot_2020-12-14 Agendas - auie(1).png ajouté
- Fichier 0001-agendas-include-event-description-in-reminders-49034.patch 0001-agendas-include-event-description-in-reminders-49034.patch ajouté
OK mais l'enchaînement des deux boutons ne rend pas forcément top.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à 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)
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: include event description in reminders (#49034)