Skip to content

Support : Préremplir le formulaire d'aide Zendesk [GEN-2536] - #6418

Merged
tonial merged 5 commits into
masterfrom
alaurent/zendesk_ticket
Aug 7, 2025
Merged

tonial merged 5 commits into
masterfrom
alaurent/zendesk_ticket

Conversation

@tonial

@tonial tonial commented Jun 26, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Permettre de faciliter la saisie d'un ticket Zendesk.

Un lien a été ajouté dans le menu déroulant Besoin d'aide du menu de gauche.
Les liens vers zendesk des différentes parties du site utilisent à présent le contexte de connexion pour pré-remplir le formulaire zendesk.

Lien vers la documentation des champs pré-remplis : https://support.zendesk.com/hc/fr/articles/4408839114522-Création-de-formulaires-de-ticket-préremplis

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial self-assigned this Jun 26, 2025
@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog. labels Jun 26, 2025
@github-actions

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@tonial
tonial force-pushed the alaurent/zendesk_ticket branch from beda521 to 5a2743a Compare June 26, 2025 05:37
@tonial tonial removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 26, 2025
@tonial
tonial force-pushed the alaurent/zendesk_ticket branch 2 times, most recently from acaacc6 to 2600a32 Compare July 16, 2025 05:19
@tonial tonial added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jul 28, 2025
@github-actions

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@tonial
tonial force-pushed the alaurent/zendesk_ticket branch 3 times, most recently from 76ec03a to 01c186a Compare July 30, 2025 06:34
@tonial
tonial requested review from rsebille and xavfernandez July 30, 2025 06:34
@tonial

tonial commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

@rsebille @xavfernandez je veux bien un avis initial sur cette PR (notamment le serializer pour le lien zendesk que je trouve moche, mais je ne sais pas si c'est vraiment super utile d'essayer de le rendre joli vu les écart entre nos enums et ceux dans zendesk)

Peut être qu'on pourrait modifier ceux dans zendesk cela dit ... 🤔

@xavfernandez

xavfernandez commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

C'est effectivement moche et plein de valeurs magiques (et j'ai donc un peu peur d'une désynchro silencieuse si on envoie de mauvais nom de champs ou des valeurs non valides) mais je n'ai pas de meilleure proposition 😬 🤷‍♂️
Et bon c'est tout de même assez localisé...

@tonial

tonial commented Jul 30, 2025

Copy link
Copy Markdown
Contributor Author

Je pense que je vais sortir la fonction moche dans un fichier distinct avec un gros commentaire pour dire "oui, c'est moche mais c'est comme ça".
Sachant qu'une désynchro n'est pas gravissime (la valeur ne sera pas prise en compte) et on n'a de toute manière pas trop moyen de tester... (sauf peut être en téléchargeant le csv des valeurs d'enum pour chaque champ, mais ça me semble pas ultra pratique à faire)

@tonial
tonial force-pushed the alaurent/zendesk_ticket branch 4 times, most recently from 8145a09 to 6297a55 Compare August 2, 2025 12:34
@tonial tonial changed the title Exploration - Ajouter un lien vers Zendesk avec du context Support - Préremplir le formulaire d'aide Zendesk Aug 3, 2025
@tonial
tonial marked this pull request as ready for review August 3, 2025 05:16
@tonial tonial changed the title Support - Préremplir le formulaire d'aide Zendesk Support - Préremplir le formulaire d'aide Zendesk [GEN-2536] Aug 3, 2025
@notion-workspace

Copy link
Copy Markdown

@tonial
tonial force-pushed the alaurent/zendesk_ticket branch 3 times, most recently from 24ba259 to 5ca0f87 Compare August 5, 2025 04:47
@tonial
tonial requested a review from EwenKorr August 5, 2025 14:35
@tonial
tonial requested a review from leo-naeka August 5, 2025 14:35
@tonial

tonial commented Aug 5, 2025

Copy link
Copy Markdown
Contributor Author

Je viens de me rendre compte que les lecteurs assignés étaient tous deux en congés 🙈
@leo-naeka @EwenKorr si vous avez le temps demain de relire cette PR vous aurez ma considération éternelle ;)

@EwenKorr EwenKorr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas comment faire mieux pour les correspondances enum/zendesk.

LGTM 👍

rel="noopener"
class="dropdown-item has-external-link"
target="_blank"
aria-label="contactez l’assistance (ouverture dans un nouvel onglet)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit : "Contacter" à l'infinitif, comme le contenu du bouton ?
Et éventuellement avec une majuscule, comme les autres aria-label

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voire Contacter le support (pour être consistant d'un écran à l'autre)

Comment thread itou/templates/gps/group_list.html Outdated
rel="noopener"
class="dropdown-item has-external-link"
target="_blank"
aria-label="contactez l’assistance (ouverture dans un nouvel onglet)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voire Contacter le support (pour être consistant d'un écran à l'autre)

Comment thread itou/utils/templatetags/support.py Outdated
Comment thread itou/utils/zendesk.py Outdated
Comment thread itou/utils/zendesk.py Outdated
Comment thread itou/utils/zendesk.py Outdated
Comment thread tests/utils/test_urls.py Outdated
Comment thread tests/utils/test_urls.py Outdated
tonial added 4 commits August 7, 2025 07:15
We can use partial instead of a lambda
One case was duplicated
use for_snapshot
We will soon add another templatetag for zendesk
@tonial
tonial force-pushed the alaurent/zendesk_ticket branch from 5ca0f87 to d6a92e3 Compare August 7, 2025 05:15
@tonial
tonial force-pushed the alaurent/zendesk_ticket branch from d6a92e3 to c85b879 Compare August 7, 2025 08:02
@tonial
tonial added this pull request to the merge queue Aug 7, 2025
Merged via the queue into master with commit b42bc07 Aug 7, 2025
14 checks passed
@tonial
tonial deleted the alaurent/zendesk_ticket branch August 7, 2025 08:29
@xavfernandez xavfernandez changed the title Support - Préremplir le formulaire d'aide Zendesk [GEN-2536] Support : Préremplir le formulaire d'aide Zendesk [GEN-2536] Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants