Project

General

Profile

Development #49034

inclure la description de l'événement dans le mail de rappel

Added by Frédéric Péters almost 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
02 December 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Related to Chrono - Development #48877: API : renvoyer la description de l'évènement sur le endpoint slot_statusFermé26 November 2020

Actions

Associated revisions

Revision b33972c3 (diff)
Added by Valentin Deniaud almost 3 years ago

agendas: include event description in reminders (#49034)

History

#1

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
#2

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

(description + autres infos sur l'événement genre l'URL)

#3

Updated by Valentin Deniaud almost 3 years ago

  • Assignee set to Valentin Deniaud
#4

Updated by Valentin Deniaud 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 ?

#6

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 ?

#7

Updated by Valentin Deniaud almost 3 years ago

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).

#8

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 ?

#9

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

et aussi, sur la description, je serais pour faire |linebreaks.

#10

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 ?

#11

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.

#13

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.

#15

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

Je trouve ça va.

#16

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

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

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)
#18

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

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

Also available in: Atom PDF