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

Track la recheche en tant que telle dans matomo #6362

Merged
merged 9 commits into from
Aug 16, 2022

Conversation

artragis
Copy link
Member

Le but de cette PR est de changer la manière dont le moteur de recherche est tracké car matomo permet d'avoir un rapport spécifique à cet usage :

search
search_report

N'ayant pas d'environnement avec la recherche installée, je n'ai pas pu tester

Contrôle qualité

Dans un environnement qui a la recherche ET matomo,
faire des recherches avec des mots différents, visitez les résultats, naviguer entre les pages.

S'assurer que :

  1. ça ne génère pas d'erreur
  2. ça envoie des données à matomo pour ce rapport

@coveralls
Copy link

coveralls commented Jul 26, 2022

Coverage Status

Coverage decreased (-0.03%) to 88.177% when pulling e8a1512 on artragis:feat/matomo_track_enhance into 06b2853 on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Ça peut être pratique comme fonctionnalité !

Je viens juste de parcourir rapidement le code, je ferai la QA plus tard en déployant sur la bêta.

zds/middlewares/matomomiddleware.py Outdated Show resolved Hide resolved
@artragis
Copy link
Member Author

Le résultat actuel c'est ça :
rapport-recherche

Si le rapport "nombre de mot clef sans résultat" est vide c'est parce que je n'arrive pas à avoir un résultat fiable ET performant pour l'instant pour compter le nombre de résultat.
L'objet "Search" qui est retourné par le sdk elastik est iterable mais ne surcharge pas __len__ ce qui impose de faire une requête count à elastic, ce qui peut être long, juste pour une stat.

Je pense avoir une astuce à base de paginator/page_count. mais pour l'instant j'ai pas eu le temps de l'implémenter.

@philippemilink
Copy link
Member

Je merge ou bien j'attends que tu tentes quelque chose pour nombre de mot clef sans résultat ?

@artragis
Copy link
Member Author

je tente un truc ce soir sur la béta, si ça marche pas on mergera

@artragis
Copy link
Member Author

après test, ça fonctionne :
mots clefs sans resultat

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

@philippemilink philippemilink merged commit 71148f7 into zestedesavoir:dev Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants