Produits Entr'ouvert » Combo

C'est une besoin spécifique à Orléans ?

• il faut nommer les placeholders de façon plus explicite (genre "à afficher si l'utilisateur n'est pas connecté", "à afficher si l'utilisateur n'est pas relié")
• le second placeholder ne semble pas placé au bon endroit pour remplacer le texte initial

Je pense nécessaire de préciser dans le libellé qu'on y parle de la cellule famille (l'info n'apparaitra autrement pas).

Ne pas prendre les noms "anonymous_user" et "unlinked_user" pour tes placeholders limités à la cellule famille.

On pourrait envisager de préfixer les libelles des placeholders des cellules avec le nom de la cellule.

Si tu penses que c'est la bonne manière de faire, et sinon tu n'aurais pas fait de ticket, le faire tout de suite ?

_~~

Quid de la migration, on va faire une mise à jour et soudainement les textes "You have to connect and link your account to a family." et " family linked to your account." vont disparaitre.

Je pense que c'est une bonne manière de faire. Mais je ne pourrais pas le faire avant la semaine prochaine (je suis en congés).

Quant à l'impacte sur l'existant il n'y a que Fontenay en prod qui a la cellule famille avec ce template par défaut et on peut facilement mettre le texte existant dans le placeholder.

Je pense que c'est une bonne manière de faire. Mais je ne pourrais pas le faire avant la semaine prochaine (je suis en congés).

Il y a une urgence particulière qui demanderait ce patch en prod avant que ça ne puisse être fait ?

Oui, Orléans pour la semaine prochaine.

Ne pas prendre les noms "anonymous_user" et "unlinked_user" pour tes placeholders limités à la cellule famille.

Pas bougé.

Quant à l'impacte sur l'existant il n'y a que Fontenay en prod qui a la cellule famille avec ce template par défaut et on peut facilement mettre le texte existant dans le placeholder.

Il faut ajouter l'info dans la page de mise à jour Publik.

Serghei Mihai a écrit :

Ok. Noms changés.

Manquent des tests, ton code a familyunlinked_user et family_unlinked_user.

Je rajouterai l'info dans la release note.

Pour que les choses soient bien claires, je parle de [[sysadmin:Mise à jour Publik]], il n'y a pas à attendre, ça doit être fait.

Avec un test.

Il n'attrape pas l'erreur du patch précédent, dont le résultat serait qu'on puisse remplir la zone depuis le /manage/ mais que le contenu n'apparaisse jamais.

Mmm; je me dis là qu'on devrait en fait zapper family_anonymous_user; il suffit sur les cellules mises dans le placeholder de paramétrer la visibilité, et ainsi le patch devient genre :

{% trans "Top content" as top_content %}
{% placeholder "family_top_content" name=top_content %}

(je pense aussi que la phrase "If you have family credentials, please..." qui reste en dur devrait être dégagée.)

Cela veut dire qu'une cellule Text mise dans le placeholder avec la visibilité "privée" s'affichera indifféremment du fait si l'usager est lié ou pas à la famille.

Je préfère garder le texte "If you have family credentials, please" car il contient le lien vers le formulaire d'appairage.

Cela veut dire qu'une cellule Text mise dans le placeholder avec la visibilité "privée" s'affichera indifféremment du fait si l'usager est lié ou pas à la famille.

Ok ça ajouterait deux lignes pour avoir render=False ici et des appels sans le render aux deux moments où ça devrait être affiché.

Je préfère garder le texte "If you have family credentials, please" car il contient le lien vers le formulaire d'appairage.

Le lien serait bien sûr à conserver. (de la même manière, on a un "Unlink", sans phrase).

Je vois, mais je trouve ça lourd à comprendre du point de vue fonctionnel.

Ok, passons. (ça pourrait s'appeler "top content for unlinked users" et être compris. Et je reste sur l'idée qu'on ne peut pas multiplier ainsi les placeholders pour la moindre condition, sans au moins s'interroger sur la présentation générale; et que partir sur deux placeholders ici compliquera les possibilités plus tard.)

<a href="{{ link_url }}" rel="popup">Link to your family.</a>

C'est l'occasion de virer le blocktrans autour, pour ne pas avoir de markup dans le .po.

Tu le mettrais pas au-dessus du "Link to your family" ?

(et taper ce dernier dans un <div> avec une classe qu'on pourrait cibler et pas de . final, pour faire ensuite correctement bouton. (quand je disais "comme le Unlink" je pensais vraiment totalement comme).)

Ok zou.