Projet

Général

Profil

Support #25192

grenoble-metropole: gérer le code de suivi des visites.

Ajouté par Serghei Mihai il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai
Version cible:
-
Début:
10 juillet 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

.

Fichiers

0001-grenoble-metropole-add-visits-tracking-code-25192.patch (1,42 ko) 0001-grenoble-metropole-add-visits-tracking-code-25192.patch Serghei Mihai, 10 juillet 2018 17:02
0001-grenoble-metropole-add-visits-tracking-code-25192.patch (1,42 ko) 0001-grenoble-metropole-add-visits-tracking-code-25192.patch Serghei Mihai, 10 juillet 2018 18:50
0001-grenoble-metropole-allow-users-visits-tracking-25192.patch (1,42 ko) 0001-grenoble-metropole-allow-users-visits-tracking-25192.patch Serghei Mihai, 12 juillet 2018 14:25
0001-grenoble-metropole-allow-users-visits-tracking-25192.patch (1,41 ko) 0001-grenoble-metropole-allow-users-visits-tracking-25192.patch Serghei Mihai, 12 juillet 2018 15:07

Demandes liées

Lié à Intégrations graphiques Publik - Development #25193: séparer le bloc avec le tracking js dans un template à partFermé10 juillet 2018

Actions

Révisions associées

Révision df80ea71 (diff)
Ajouté par Serghei Mihai il y a presque 6 ans

grenoble-metropole: allow users visits tracking (#25192)

Historique

#1

Mis à jour par Serghei Mihai il y a presque 6 ans

#2

Mis à jour par Frédéric Péters il y a presque 6 ans

Je pense important de ne pas trop charger le code python (ici plutôt juste taper un {% block ... %} dont le contenu réel sera contrôlé par theme.html.

Et également tout aussi important de ne pas dupliquer et à coup sûr rater les évolutions, ici j'imagine qu'on pourrait avoir includes/tracking.html, et l'inclure dans le templates/theme.html principal et dans cette variation grenoble (et hautes-alpes-2018, aussi).

#3

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Lié à Development #25193: séparer le bloc avec le tracking js dans un template à part ajouté
#4

Mis à jour par Serghei Mihai il y a presque 6 ans

Effectivement, ça evite la duplication inutile du code.

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

Évitons d'appeler ça "tracking code", qui correspond déjà au "code de suivi" des demandes wcs.

#7

Mis à jour par Frédéric Péters il y a presque 6 ans

Je parlais de {% block tracking-code %}.

#8

Mis à jour par Serghei Mihai il y a presque 6 ans

Ok, un bête {% block tracking %}.

#9

Mis à jour par Frédéric Péters il y a presque 6 ans

(manque le patch)

#11

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de Solution proposée à Solution validée

Ok.

#12

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit df80ea71e596b88c43a3a2e71ac90d21933f9920 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jul 10 16:59:53 2018 +0200

    grenoble-metropole: allow users visits tracking (#25192)
#13

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF