Development #58182
braine-l'alleud, aller chercher les délbérations communales via le connecteur plone
0%
Fichiers
Demandes liées
Révisions associées
braine-l-alleud: update padding of deliberation tile selector (#58182)
Historique
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a plus de 2 ans
Pour l'utiliser, il faut #58181, puis instancier une nouvelle instance du connecteur Plone avec la configuration donnée ici :
https://dev.entrouvert.org/issues/51764#note-14
comme c'est en partie fait sur la recette.
https://passerelle-braine-l-alleud.test.entrouvert.org/plone-restapi/iadelib/
(le paramètre path
peut être remplacé par le champ uri
de l'objet Query du connecteur)
Mis à jour par Serghei Mihai il y a plus de 2 ans
- Lié à Development #56545: braine l’alleud, adapter la cellule Actualité pour aller chercher les contenus sur plone. ajouté
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier Screenshot_2021-11-08 Portail - tableau de bord.png Screenshot_2021-11-08 Portail - tableau de bord.png ajouté
- Fichier 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch ajouté
Rebasé.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Je préférerais qu'il n'y ait pas de paramètres, que ça vienne d'une/deux variables posées dans hobo. La raison c'est que les paramètres sont copiés une fois la mise en favori de la tuile et ça fait donc qu'il n'est pas possible de les changer s'ils sont paramétrés au niveau de la cellule.
Si tu veux changer la durée de cache ok mais tape ça dans un commit spécifique.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch ajouté
- Fichier Screenshot_2021-11-09 Mes super démarches - Braine-l_Alleud 2.png Screenshot_2021-11-09 Mes super démarches - Braine-l_Alleud 2.png ajouté
Fait.
J'ai mis 2 variables pour forcer l'utilisation du connecteur, parce que le format retourné n'est pas plus le même (le tableau "items" devient "data").
Mis à jour par Frédéric Péters il y a plus de 2 ans
J'ai mis 2 variables pour forcer l'utilisation du connecteur, parce que le format retourné n'est pas le même
Je ne comprends pas comment écrire
"url" : "{{ passerelle_url }}plone-restapi/{{ braine_deliberations_communales_connector_slug }}/q/{{ braine_deliberations_communales_query_slug }}/
plutôt que
"url" : "{{ braine_deliberations_communales_json_url|safe }}"
change le format des données reçues, à partir du moment où la chaine qui contient l'URL au final contient le même contenu.
Il faut garder braine_deliberations_communales_json_url et dans hobo il faudra en changer la valeur vers https://whatever/plone-restapi/whatever.
Je ne capte pas la modif à static/braine-l-alleud/_custom.scss.
Bref, pour moi il y a juste templates/variants/braine-l-alleud/combo/json/deliberations-communales.html à inclure dans ce patch.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier Screenshot_2021-11-09 Mes super démarches - Braine-l_Alleud 2(1).png Screenshot_2021-11-09 Mes super démarches - Braine-l_Alleud 2(1).png ajouté
- Fichier 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch ajouté
Il faut garder braine_deliberations_communales_json_url et dans hobo il faudra en changer la valeur vers https://whatever/plone-restapi/whatever.
J'avais pensé que l'avantage d'avoir une unique variable était de permettre de rebasculer sur l'URL actuelle qui consiste à interroger plone directement. Comme ça ne fonctionnera plus puisque la cellule ne va pas chercher les enregistrements dans le même tableau, je pensais explicitement ne pas laisser cette alternative.
Je ne capte pas la modif à static/braine-l-alleud/_custom.scss.
C'était pour avoir le même rendu que sur les tuilles actualités et événements. (cf copie d'écran)
Bref, pour moi il y a juste templates/variants/braine-l-alleud/combo/json/deliberations-communales.html à inclure dans ce patch.
Fait.
Mis à jour par Frédéric Péters il y a plus de 2 ans
permettre de rebasculer sur l'URL actuelle
non.
C'était pour avoir le même rendu que sur les tuilles actualités et événements. (cf copie d'écran)
Ok tu peux explicitement dire "aujourd'hui le rendu sur la page "mon compte" de l'usager n'est pas correct, le texte est collé au bord. (mais ça n'a pas de rapport avec ce ticket, fais-en a minima un commit sépara).
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Fichier 0002-braine-l-alleud-update-pading-of-deliberation-tile-s.patch 0002-braine-l-alleud-update-pading-of-deliberation-tile-s.patch ajouté
- Fichier 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch 0001-braine-l-alleud-get-deliberations-communales-from-pl.patch ajouté
fais-en a minima un commit séparé
J'ai fait ça (j'ai bien compris que j'aurais du le poser explicitement dans un autre ticket, désolé).
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
padding avec deux d, dans le message de commit.
Mis à jour par Nicolas Roche il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 525128ee0ce516f01114a7a4abd8c131dcecbffa Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue Nov 9 15:08:45 2021 +0100 braine-l-alleud: update padding of deliberation tile selector (#58182) commit f41782209f3c23f1e6e2d8f20ecf0431ae24b95f Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Tue Nov 9 13:26:03 2021 +0100 braine-l-alleud: get deliberations-communales from plone (#58182)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
braine-l-alleud: get deliberations-communales from plone (#58182)