Projet

Général

Profil

Development #49034

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

Ajouté par Frédéric Péters il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
02 décembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Lié à Chrono - Development #48877: API : renvoyer la description de l'évènement sur le endpoint slot_statusFermé26 novembre 2020

Actions

Révisions associées

Révision b33972c3 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

agendas: include event description in reminders (#49034)

Historique

#1

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

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)

#3

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Assigné à mis à Valentin Deniaud
#4

Mis à jour par Valentin Deniaud 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 ?

#6

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 ?

#7

Mis à jour par Valentin Deniaud il y a plus de 3 ans

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

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 ?

#9

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.

#10

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 ?

#11

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.

#13

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.

#15

Mis à jour par Frédéric Péters il y a plus de 3 ans

Je trouve ça va.

#16

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#17

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

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

Formats disponibles : Atom PDF