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.

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

Matomo track phase2 #6387

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

Conversation

artragis
Copy link
Member

@artragis artragis commented Sep 5, 2022

Je continue à explorer matomo et son fonctionnement.

Cette PR a pour but d'obtenir deux informations :

  • on qualifie le fait qu'obtenir les epub et pdf (ce sont les seuls publics) sont un téléchargement et non une simple impression
  • on essaie de voir l'efficacité des suggestions dans les tutos et article

Autant je suis assez sûr de moi pour ce que va donner la première partie, autant j'ai plus de doute pour la seconde.
En effet contrairement au téléchargement, il n'y a rien dans matomo pour assurer ce genre de choses. Ou alors il faut utiliser le plugin payant "marketing campaign" qui inclut les UTM (https://fr.wikipedia.org/wiki/Param%C3%A8tres_UTM) (MTM dans le langage matomo, UTM c'est google à la base) ce qui permettrait de gérer facilement ce genre de choses.

Note: je propose cette PR car ça me plait de travailler sur matomo, si vous trouvez que ce travail va trop loin ou n'est pas souhaitable, on peut clore rapidement la PR

Contrôle qualité

Sur un site qui a matomo

  • publier un contenu
  • publier les exports
  • en télécharger un
  • Dans matomo, vérifier la page comportement > téléchargements

Mettre des suggestions
naviguer sur le site au travers de ces suggestions
regarder si c'est rapporté quelque part dans matomo.

@coveralls
Copy link

coveralls commented Sep 5, 2022

Coverage Status

Coverage decreased (-0.03%) to 88.26% when pulling 47f409e on artragis:matomo_track_phase2 into a40bc42 on zestedesavoir:dev.

@philippemilink philippemilink added this to En développement in Suivi des PR via automation Sep 5, 2022
@philippemilink philippemilink moved this from En développement to En attente de QA in Suivi des PR Sep 5, 2022
@Arnaud-D
Copy link
Contributor

Arnaud-D commented Sep 8, 2022

Pour la QA, c'est quelque chose à tester sur la bêta ?

@artragis
Copy link
Member Author

artragis commented Sep 8, 2022

ça serait plus simple. Sinon y'a toujours la possibilité d'utiliser la dev key de matomo.

@philippemilink
Copy link
Member

J'ai tenté de déployer sur la bêta, et ça râle :

AttributeError: module 'requests' has no attribute 'GET'
  File "zds/middlewares/matomomiddleware.py", line 135, in process_response
    self.matomo_track(request)
  File "zds/middlewares/matomomiddleware.py", line 103, in matomo_track
    "get_params": dict(**requests.GET),

@philippemilink philippemilink moved this from En attente de QA to Modification demandée in Suivi des PR Dec 6, 2022
@Situphen
Copy link
Member

Il serait bien d'ajouter quelques commentaires dans zds/middlewares/matomomiddleware.py pour préciser quelle partie est liée aux statistiques en générale, laquelle est pour la recherche et laquelle est pour les exports.

@Situphen Situphen added the S-Zombie Ticket ou PR oubliée label Mar 9, 2024
@Situphen Situphen moved this from Modification demandée to En attente de reprise in Suivi des PR Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Zombie Ticket ou PR oubliée
Projects
Status: En attente de reprise
Suivi des PR
  
En attente de reprise
Development

Successfully merging this pull request may close these issues.

None yet

5 participants