Projet

Général

Profil

Bug #53694

lille, bloc de connexion/info utilisateur

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

#52275 dit "lille: isolate user-info template" et ça a cassé le rendu attendu, il n'y a plus d'espace de connexion depuis.


Fichiers

Révisions associées

Révision 450eb8c5 (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

lille: be explicit about top links as the value changed (#53694)

Révision 76a9c5da (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

moselle: be explicit about top links as the value changed (#53694)

Historique

#2

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

Apparemment lié d'une manière ou d'une autre au déploiement; de manière générale quelque chose comme ça :

--- a/static/lille/config.json
+++ b/static/lille/config.json
@@ -1,7 +1,6 @@
 {
   "label": "Lille",
   "variables": {
-    "include_top_links": false,
     "pwa_display": "standalone",
     "email_header_img": "img/email-logo.png",
     "theme_color": "#e1001a" 

il ne faut pas faire il n'y a rien qui va capter que la variable a disparu, même si après on pense à redéployer. (ce qu'on ne fait de toute façon pas sauf si ça nous est dit explicitement).

#3

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

#4

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

  • Assigné à changé de Thomas Jund (congés, retour le 29/04) à Frédéric Péters

(appliqué à peu près à l'arrache)

#5

Mis à jour par Thomas Noël il y a presque 3 ans

  • Statut changé de Solution proposée à Solution validée
  • Assigné à changé de Frédéric Péters à Benjamin Dauvergne
#6

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

  • Statut changé de Solution validée à Résolu (à déployer)
  • Assigné à changé de Benjamin Dauvergne à Frédéric Péters
commit 450eb8c528551a6961fd67365a81d7b2e878e41a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon May 3 18:19:37 2021 +0200

    lille: be explicit about top links as the value changed (#53694)
#7

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

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

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

J'avais pourtant testé ces thèmes :/.
Je ne comprends toujours pas le fonctionnement des variables posées dans config.json et dupliqués dans les différents hobo.json et je veux bien une explication sur pourquoi il faut explicitement ajouter

    "include_top_links": true;

sachant que c'est la valeur par défaut.
Ou encore, pourquoi c'est nécessaire dans le cas d'un thème qUI avait auparavant déclaré cette variable en "false" (plutôt que de simplement supprimer ladite variable pour quelle retrouve sa valeur par défaut.

#9

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

Ou encore, pourquoi c'est nécessaire dans le cas d'un thème qUI avait auparavant déclaré cette variable en "false" (plutôt que de simplement supprimer ladite variable pour quelle retrouve sa valeur par défaut.

L'affaire c'est pas juste "tiens un fichier je regarde dedans" mais bien plus complexe (trop on est sans doute d'accord); la variable est enregistrée dans hobo, communiquée/propagée, mais pour que ça ait lieu il faut qu'elle soit définie. Si elle n'est pas définie, il ne se passe rien. S'il ne se passe rien l'ancien état est conservé.

Formats disponibles : Atom PDF