Development #25820
permettre de surcharger le champ "summary" dans la vue "ics" d'une réservation
0%
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.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-allow-booking-s-ics-summary-field-overriding-25820.patch 0001-allow-booking-s-ics-summary-field-overriding-25820.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 5 ans
Tu peux noter dans ce ticket technique la formulation fonctionnelle de la demande ?
Mis à jour par Frédéric Péters il y a plus de 5 ans
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 ?
Mis à jour par Serghei Mihai il y a plus de 5 ans
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..."
Mis à jour par Thomas Noël il y a plus de 5 ans
(y'avait pas l'idée d'envoyer un ics d'annulation ?)
Mis à jour par Frédéric Péters il y a plus de 5 ans
Et ? Sans doute que je ne vois pas bien ce qui s'entend par "ics d'annulation" et le rapport avec ce patch.
Mis à jour par Serghei Mihai il y a plus de 5 ans
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.
Mis à jour par Frédéric Péters il y a plus de 5 ans
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.
Mis à jour par Serghei Mihai il y a plus de 5 ans
Yep.
Et plus tard rajouter aussi la description, le lieu, etc.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à En cours
- Patch proposed changé de Oui à Non
On s'accorde donc pour éviter ce hack spécifique ics.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch 0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch ajouté
- Patch proposed changé de Non à Oui
Attribut dédié pour l'affichage à l'usager, utilisé en priorité dans le "summary" de l'ics généré.
Mis à jour par Frédéric Péters il y a environ 5 ans
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.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch 0001-agendas-add-new-booking-s-label-attribute-for-user-p.patch ajouté
- Statut changé de En cours à Solution proposée
- Assigné à mis à Serghei Mihai
Attribut rénommé pour plus de clarté.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
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
?
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Serghei Mihai il y a environ 5 ans
Benjamin Dauvergne a écrit :
Ack, mais j'aime pas trop le
user_display_label
juste aprèsuser_name
on dirait qu'il y a un rapport, un plus simpleics_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.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Statut changé de Solution validée à 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
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
label_for_user
aurait été mieux alors, qu'on fasse le lien avec label plus haut en les groupant ensemble, tant pis.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agendas: add new booking's label attribute for user presentation purposes (#25820)
Use the new label in ICS summary