Produits Entr'ouvert » Hobo

Rapidement en testant.

S'il n'y a pas de config définie pour Matomo, ne pas exposer le bouton "automatique" (qui ne va pas fonctionner).

Dans le menu sur l'accueil le libellé est toujours "Matomo", il faudrait là aussi que ça soit "User tracking".

Dans le settings.py, il faut garder la partie execfile(local_settings_file) en dernier, pour permettre à la configuration de passer au-dessus des valeurs par défaut.

Comme on a une bonne valeur par défaut, on peut lui faire confiance, et faire settings.MATOMO_SERVER plutôt que passer par getattr().

Le code js inséré contient <!-- Matomo --> et <!-- End Matomo Code --> qui ne sont pas ok pour du js.

L'affichage de "Automatic mode" sans autre forme de procès est curieuse; une fois configuré je ferais disparaitre l'encart de description Matomo; j'afficherais juste (en infonotice également) le "Automatic mode" (en premier, avec le "excellent respect" dessous). Et pareil pour le mode manuel.

J'ai poussé deux commits d'ajustements dans la branche,

• pour réagencer/reformuler un peu les textes d'explication et libellés; (ce qui va demander ajustement à la traduction)
• pour traduire un bout de js.

Le truc supplémentaire que je verrais bien, mais disons qu'on se le garde pour un ticket plus tard, c'est améliorer le compute_cnil_acknowledgment_level, pour 1) retourner "unknown" quand on n'a pas trop d'idée, 2) détecter d'autres hostiles (mais à part GL où à un moment il a été question de xiti, je n'ai pas de liste).

Voilà, si ça te va tu peux prendre mes deux commits et les intégrer dans les tiens (git rebase -i, fixup) et il me semble que ça sera bon.

Alors perso, je vais dans le hobo/ avec les sources et je fais ../manage.py makemessages -l fr.

Je l'ai compté dans l'annonce des mises à jour de la prochaine version, si on reporte, je retire, pas de soucis.

et tester sur piwik (la version actuellement en prod).

Pas particulièrement utile, ce qu'il faut maintenant c'est avoir matomo en prod (je viens de faire #32017).

Je l'ai compté dans l'annonce des mises à jour de la prochaine version, si on reporte, je retire, pas de soucis.

Et à réfléchir je vais attendre qu'on ait un matomo de prod pour annoncer ça.

Je ne pige pas trop; il y a une nouvelle vue "enabling", mais de l'accueil de hobo, on va toujours sur le matomo/.

De ce patch je ne vois pas quel pouvait être le problème et comment cette vue change quoique ce soit. ?

Dans un commentaire ici, il faudrait que tu décrives les différentes situations que la page matomo/ peut rencontrer, et pour chacune ce qu'elle doit afficher, façon :

• aucune variable définie : boutons pour activer (automatiquement ou pas)
• un tracking_js défini : ...
• un cnil_compliant_tracking_js
  • dont on reconnait le contenu (genre le js contient ~ MATOMO_SERVER['url']) : ...
  • dont on ne reconnait pas le contenu : ...
• etc.

• Excellent: on detecte la fonction de gestion des cookie préconisée par la CNIL
• Bon: on ne détecte pas 'google'
• Mauvais: on détecte 'google'

Si le niveau est Bon ou Excellent : on stocke le code Javascript dans 'cnil_compliant_tracking_js' et donc on l'expose en l'état sur les front-ends.

Si le niveau est Mauvais : on le stocke le JS dans 'tracking_js', ce qui a pour effet d'afficher un bandeau à l'utilisateur afin qu'il accepte le code JS. S'il ne l'accepte pas, alors le code n'est pas ajouté aux pages front-ends.

• WS: settings.json est configuré pour permettre d'administrer un serveur matomo
• ENABLED: concatenation de 'tracking_js' + 'cnil_compliant_tracking_js' non vide
• MODE_AUTO: présence d'une url de login Matomo pour le client

voici le comportement de la vue principale :

ENABLED WS AUTO
non     ?    ?  -> topo sur les outils de mesure d'audiance
                   affiche 'Support is currently disabled.'
                   présence du bouton: 'Manual Configuration'
non     non  ?  -> informe que Matomo est recommandé par la CNIL
non     oui  ?  -> informe que l'on peut configurer Matomo
                   deuxième bouton: 'Automatic Configuration'

oui     ?    ?  -> présence du bouton 'Disable'
                   affiche un niveau de respect des droits de l'usager
oui     ?   non -> affiche 'Manual Configuration.'
                   affiche le formulaire pour remplir le code Javascript
oui     ?   oui -> deuxième bouton 'Open visit tracking dashboard'
                   affiche 'Automatic Configuration.'

Le 3 derniers patchs proposent différentes solutions au problème suivant :
Proposer le formulaire pour remplir le code Javascript lorsque l'on valide, après avoir appuyé sur le bouton 'Manual Configuration'. (ie: ENABLED reste à 'non' tant qu'aucun code Javascript n'est ajouté)

Parfait je vois; solution : dans la page d'activation du mode "manuel", déjà demander le js.

Je virerais le "Are you sure you want to enable manual user tracking support?" ainsi que le help_text=_('See...; aussi, l'attribut "size" n'existe pas pour les textarea, il existe "rows" et "cols" mais ici je n'en ferais rien (parce que ça doit être affiché aussi bien en popup qu'en pleine page), et j'ajouterais simplement aux CSS :

textarea#id_tracking_js {
       width: 100%;
}