Project

General

Profile

Development #25820

permettre de surcharger le champ "summary" dans la vue "ics" d'une réservation

Added by Serghei Mihai about 1 year ago. Updated 7 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
24 Aug 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Le besoin est de pouvoir surchager le sommaire dans l'ICS au lieu d'utiliser celui qui a été posé lors de la création du rendez-vous.
Si lors de la résa on met par exemple le libellé du rendez-vous, par exemple "rdc cni" ou "rdv passeport", dans l'ics envoyé à l'usager il peut être souhaité que le sommaine soit un peu plus orienté usager, par exemple "Votre rendez-vous passeport à la mairie principale",

Au lieu de l'actuel:

vevent.add('summary').value = self.label

permettre de poser autre chose via les variables GET.

0001-allow-booking-s-ics-summary-field-overriding-25820.patch View (2.4 KB) Serghei Mihai, 24 Aug 2018 03:38 PM

0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch View (5.43 KB) Serghei Mihai, 14 Mar 2019 03:14 PM

0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch View (5.58 KB) Serghei Mihai, 14 Mar 2019 04:29 PM

Associated revisions

Revision 0cfa22db (diff)
Added by Serghei Mihai 7 months ago

agendas: add new booking's label attribute for user presentation purposes (#25820)

Use the new label in ICS summary

History

#2 Updated by Serghei Mihai about 1 year ago

#3 Updated by Frédéric Péters about 1 year ago

Tu peux noter dans ce ticket technique la formulation fonctionnelle de la demande ?

#4 Updated by Serghei Mihai about 1 year ago

  • Description updated (diff)

#5 Updated by Frédéric Péters about 1 year ago

Si lors de la résa on met par exemple le libellé du rendez-vous, par exemple "rdc cni" ou "rdv passeport", dans l'ics envoyé à l'usager il peut être souhaité que le sommaine soit un peu plus orienté usager, par exemple "Votre rendez-vous passeport à la mairie principale",

Pourquoi ne pas mettre un libellé correct dès le début ?

#6 Updated by Serghei Mihai about 1 year ago

Puisque le libellé tel que posé lors de la résa et affiché dans le backoffice aux agents est parlant aux agents: "Rendez-vous passeport - Monsieur Dupont" et à l'usager on souhaite pouvoir afficher autre chose dans le sommaire: "Votre rendez-vous passeport..."

#7 Updated by Thomas Noël about 1 year ago

(y'avait pas l'idée d'envoyer un ics d'annulation ?)

#8 Updated by Serghei Mihai about 1 year ago

Il y avait ça aussi comme demande.

#9 Updated by Frédéric Péters about 1 year ago

Et ? Sans doute que je ne vois pas bien ce qui s'entend par "ics d'annulation" et le rapport avec ce patch.

#10 Updated by Serghei Mihai about 1 year ago

Aucun.

Mais dans un futur/potentiel .ics permettant d'annuler un rendez-vous le titre pour être "Annulation de votre rendez-vous CNI".
Bref, la demande fonctionnelle est de pouvoir envoyer un autre summary à l'usager que celui qui est affiché à l'agent dans le backoffice.

#11 Updated by Frédéric Péters about 1 year ago

Mais imaginons donc une cellule qui affiche à l'usager la liste de ses rendez-vous à venir; on y voudra aussi cette info. Si on veut attacher à un rendez-vous un libellé supplémentaire pour l'affichage à l'usager, il me semble que ça doit passer par une colonne supplémentaire au modèle, pas par un hack spécifique ics.

#12 Updated by Serghei Mihai about 1 year ago

Yep.
Et plus tard rajouter aussi la description, le lieu, etc.

#13 Updated by Frédéric Péters about 1 year ago

  • Status changed from Solution proposée to En cours
  • Patch proposed changed from Yes to No

On s'accorde donc pour éviter ce hack spécifique ics.

#14 Updated by Serghei Mihai 7 months ago

Attribut dédié pour l'affichage à l'usager, utilisé en priorité dans le "summary" de l'ics généré.

#15 Updated by Frédéric Péters 7 months ago

De la relecture j'ai créé #31421.

À part ça je taperais quand même un verbose_name sur le champ, parce que posé là sous user_name ça donne vraiment l'impression que ça pourrait contenir un autre attribut de l'utilisateur.

#16 Updated by Serghei Mihai 7 months ago

Attribut rénommé pour plus de clarté.

#17 Updated by Benjamin Dauvergne 7 months ago

Ack, mais j'aime pas trop le user_display_label juste après user_name on dirait qu'il y a un rapport, un plus simple ics_label ?

#18 Updated by Benjamin Dauvergne 7 months ago

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

#19 Updated by Serghei Mihai 7 months ago

Benjamin Dauvergne a écrit :

Ack, mais j'aime pas trop le user_display_label juste après user_name on dirait qu'il y a un rapport, un plus simple ics_label ?

Comme échangé plus haut avec Frédéric l'idéé est que ce champ serve un jour à l'affichage à l'usager la liste des ses rendez-vous dans une cellule. Donc pas que pour produire l'ics.

#20 Updated by Serghei Mihai 7 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 0cfa22dbc0bd769a3946f016ee2eb78cf7bd5f80 (HEAD -> master, origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Mar 14 14:13:05 2019 +0100

    agendas: add new booking's label attribute for user presentation purposes (#25820)

    Use the new label in ICS summary

#21 Updated by Benjamin Dauvergne 7 months ago

label_for_user aurait été mieux alors, qu'on fasse le lien avec label plus haut en les groupant ensemble, tant pis.

#22 Updated by Serghei Mihai 7 months ago

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

Also available in: Atom PDF