Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEATURE] Matomo - Envoyer les events liés à l'utilisation du Expand (toggle) (PIX-15774) #10889

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

[FEATURE] Matomo - Envoyer les events liés à l'utilisation du Expand (toggle) (PIX-15774) #10889

wants to merge 3 commits into from
+154 −8

Conversation

dianeCdrPix
Copy link
Contributor

@dianeCdrPix dianeCdrPix commented Dec 23, 2024 •
edited by er-lim
Loading

🎁 Proposition

Ajouter une fonction dans Passage pour appeler le service metrics et envoyer des informations à Matomo sur le click d'un Expand

🧦 Remarques

  • Pour le futur : refacto du passage et de ses nombreuses méthodes ?
  • Utilisation de l'événement toggle comme mis dans la doc de la balise details
  • Pour les tests, nous avons dû simuler un clic sur la balise summary sinon le contenu ne s'affiche pas.

🎅 Pour tester (en local)

  • Aller sur le didacticiel Modulix /modules/didacticiel-modulix/details
  • Afficher la console navigateur
  • Cliquer sur l'élément Expand en dessous des flashcards
  • Vérifier que l'event Matomo dans la console est bien envoyé 🎉

@dianeCdrPix dianeCdrPix requested a review from a team as a code owner December 23, 2024 11:12
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@dianeCdrPix dianeCdrPix force-pushed the pix-15774-add-matomo-event-for-expand branch 2 times, most recently from 9a2ba7a to 4c6e4c6 Compare December 23, 2024 13:44
@er-lim er-lim changed the title [FEATURE] Matomo - Envoyer events liés à l'utilisation du Expand (click) (PIX-14973)(PIX-15774) [FEATURE] Matomo - Envoyer events liés à l'utilisation du Expand (toggle) (PIX-14973)(PIX-15774) Dec 23, 2024
@er-lim er-lim changed the title [FEATURE] Matomo - Envoyer events liés à l'utilisation du Expand (toggle) (PIX-14973)(PIX-15774) [FEATURE] Matomo - Envoyer events liés à l'utilisation du Expand (toggle) (PIX-15774) Dec 23, 2024
@dianeCdrPix @er-lim
feat(mon-pix): add onExpandToggle method to expand element
10e6919 
Co-authored-by: Eric Lim <eric.lim@pix.fr>
@er-lim er-lim force-pushed the pix-15774-add-matomo-event-for-expand branch from 4c6e4c6 to f2b731a Compare December 23, 2024 15:31
@er-lim er-lim force-pushed the pix-15774-add-matomo-event-for-expand branch from f2b731a to 10a0407 Compare December 23, 2024 15:34
@er-lim er-lim changed the title [FEATURE] Matomo - Envoyer events liés à l'utilisation du Expand (toggle) (PIX-15774) [FEATURE] Matomo - Envoyer les events liés à l'utilisation du Expand (toggle) (PIX-15774) Dec 23, 2024
@dianeCdrPix dianeCdrPix marked this pull request as draft December 24, 2024 10:40
@dianeCdrPix dianeCdrPix marked this pull request as ready for review December 24, 2024 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@er-lim er-lim

@dianeCdrPix dianeCdrPix

Labels
team-devcomp
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dianeCdrPix @pix-bot-github @er-lim