WIP: toulouse-2022: add json cell to display maelis catalog (#73851) #142

Closed
nroche wants to merge 4 commits from wip/73851-toulouse-2022-activity-catalog into main
Owner

Premier jet, parce que les critères seront certainement redéfinis.
Le patch va de paire avec #73730.

Une cellule avec le même JS mais qui n'utilise pas les vraies données et qui sera peut-être plus simple à relire est posée ici :
https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=A-2,B-1
Si on clique sur le nom du critère (cyan, à gauche), ça dé-sélectionne toutes les valeurs cochées pour ce critère.
Si on clique sur une valeur (bleu, à droite) ça active/désactive le critère en question.
L'algo fait :

  • un ET sur les critères (ex: si je choisis un type d'activité et un lieu qui sont incompatibles alors rien n'est affiché)
  • un OU sur les valeurs de chaque critère (ex: je peux choisir plusieurs tranches d'âges et voir plus d'activités s'afficher)
    Aussi, on peut arriver sur la page en passant des critères déjà activés dans la query string.
    ex: ?q=type-LOI_VAC,age-0,age-1,age-3,age-4,age-5,place-A10053179604
Premier jet, parce que les critères seront certainement redéfinis. Le patch va de paire avec #73730. Une cellule avec le même JS mais qui n'utilise pas les vraies données et qui sera peut-être plus simple à relire est posée ici : https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=A-2,B-1 Si on clique sur le nom du critère (cyan, à gauche), ça dé-sélectionne toutes les valeurs cochées pour ce critère. Si on clique sur une valeur (bleu, à droite) ça active/désactive le critère en question. L'algo fait : * un ET sur les critères (ex: si je choisis un type d'activité et un lieu qui sont incompatibles alors rien n'est affiché) * un OU sur les valeurs de chaque critère (ex: je peux choisir plusieurs tranches d'âges et voir plus d'activités s'afficher) Aussi, on peut arriver sur la page en passant des critères déjà activés dans la query string. ex: ?q=type-LOI_VAC,age-0,age-1,age-3,age-4,age-5,place-A10053179604
Owner

Je pense qu'l faudrait détailler ce qui serait graphiquement attendu et faire intervenir @tjund

Je pense qu'l faudrait détailler ce qui serait graphiquement attendu et faire intervenir @tjund
First-time contributor

Fred,

Compte tenu des délais (la livraison était prévue pour le 30/01) je pense qu'il faut phaser et dissocier cette mise en place technique et la mise en place graphique (sauf si ThomasJ peut intervenir rapidement = tout de suite).

Après déploiement j'ai de la conception de page et de démarche, les métiers de Toulouse doivent jouer avec etc. Le tout doit être montré et utilisé le 20/02 en formation agent.

Fred, Compte tenu des délais (la livraison était prévue pour le 30/01) je pense qu'il faut phaser et dissocier cette mise en place technique et la mise en place graphique (sauf si ThomasJ peut intervenir rapidement = tout de suite). Après déploiement j'ai de la conception de page et de démarche, les métiers de Toulouse doivent jouer avec etc. Le tout doit être montré et utilisé le 20/02 en formation agent.
Owner

Nico, tu peux coller des captures d'écran de ce que ça donne chez toi en local?

Aussi, je pense qu'il serait préférable que les feuilles de style aillent dans le fichier style.css et le JS dans extra.js.

Nico, tu peux coller des captures d'écran de ce que ça donne chez toi en local? Aussi, je pense qu'il serait préférable que les feuilles de style aillent dans le fichier style.css et le JS dans extra.js.
Owner

cf https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=A-2,B-1 donné par Nico pour voir ce que ça donne, bien sûr ça gagnerait d'avoir des textes s'approchant davantage de la réalité pour qu'on comprenne vraiment de quoi on parle.

Compte tenu des délais (la livraison était prévue pour le 30/01) je pense qu'il faut phaser et dissocier cette mise en place technique et la mise en place graphique (sauf si ThomasJ peut intervenir rapidement = tout de suite).

Si ce n'est pas ThomasJ ça peut être quelqu'un d'autre (genre Serghei là qui se manifeste), voire moi, mais en l'état pour moi c'est trop brut/moche avec un risque trop grand que les agents bloquent dessus.

cf https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=A-2,B-1 donné par Nico pour voir ce que ça donne, bien sûr ça gagnerait d'avoir des textes s'approchant davantage de la réalité pour qu'on comprenne vraiment de quoi on parle. > Compte tenu des délais (la livraison était prévue pour le 30/01) je pense qu'il faut phaser et dissocier cette mise en place technique et la mise en place graphique (sauf si ThomasJ peut intervenir rapidement = tout de suite). Si ce n'est pas ThomasJ ça peut être quelqu'un d'autre (genre Serghei là qui se manifeste), voire moi, mais en l'état pour moi c'est trop brut/moche avec un risque trop grand que les agents bloquent dessus.
Owner

Vu avec Stef depuis buroLyon. L'objectif est d'avoir un proto non designé pour terminer l'aspect fonctionnel,
Je suis ok pour reprendre ensuite l'UI de tout ça, (une UI et un code qui pourra être récupéré pour Publik famille sur un fonctionnement similaire)

Vu avec Stef depuis buroLyon. L'objectif est d'avoir un proto non designé pour terminer l'aspect fonctionnel, Je suis ok pour reprendre ensuite l'UI de tout ça, (une UI et un code qui pourra être récupéré pour Publik famille sur un fonctionnement similaire)
Owner

Vu avec Stef depuis buroLyon. L'objectif est d'avoir un proto non designé pour terminer l'aspect fonctionnel,
Je suis ok pour reprendre ensuite l'UI de tout ça, (une UI et un code qui pourra être récupéré pour Publik famille sur un fonctionnement similaire)

Alors c'est ok pour moi, mais je te laisse valider le patch.

> Vu avec Stef depuis buroLyon. L'objectif est d'avoir un proto non designé pour terminer l'aspect fonctionnel, > Je suis ok pour reprendre ensuite l'UI de tout ça, (une UI et un code qui pourra être récupéré pour Publik famille sur un fonctionnement similaire) Alors c'est ok pour moi, mais je te laisse valider le patch.
nroche force-pushed wip/73851-toulouse-2022-activity-catalog from dc171bb10f to b930a8a025 2023-01-26 16:11:09 +01:00 Compare
Author
Owner

A mettre en relation avec ce qui a été proposé par Stéphane :
https://portail-parsifal.test.entrouvert.org/mes-activites/

Il manque par exemple le faite de pouvoir plier/déplier les résultats pour afficher le détail de l'activité (parce qu'on a pas ça dans Maélis, jusqu'à présent).

A mettre en relation avec ce qui a été proposé par Stéphane : https://portail-parsifal.test.entrouvert.org/mes-activites/ Il manque par exemple le faite de pouvoir plier/déplier les résultats pour afficher le détail de l'activité (parce qu'on a pas ça dans Maélis, jusqu'à présent).
nroche force-pushed wip/73851-toulouse-2022-activity-catalog from b930a8a025 to 650972f581 2023-01-27 18:29:02 +01:00 Compare
Author
Owner

bien sûr ça gagnerait d'avoir des textes s'approchant davantage de la réalité pour qu'on comprenne vraiment de quoi on parle.

J'ai posé le code dans une cellule proto qui pointe sur le jeux de données actuel
https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=type-LOI_VAC,public-3,public-4,public-5,day-1,day-2

c'est trop brut/moche avec un risque trop grand que les agents bloquent dessus.

Ça reste moche mais j'ai travaillé les résultats pour mieux coller au prototype de Stéphane.

> bien sûr ça gagnerait d'avoir des textes s'approchant davantage de la réalité pour qu'on comprenne vraiment de quoi on parle. J'ai posé le code dans une cellule proto qui pointe sur le jeux de données actuel https://portail-parsifal.test.entrouvert.org/proto-catalogue-des-activites-nico/?q=type-LOI_VAC,public-3,public-4,public-5,day-1,day-2 > c'est trop brut/moche avec un risque trop grand que les agents bloquent dessus. Ça reste moche mais j'ai travaillé les résultats pour mieux coller au prototype de Stéphane.
Owner

J'ai du reprendre le code de la pagination depuis combo, pour gérer les éléments masqués dans la pagination.

Dépend de https://dev.entrouvert.org/issues/74882

J'ai du reprendre le code de la pagination depuis combo, pour gérer les éléments masqués dans la pagination. Dépend de https://dev.entrouvert.org/issues/74882
csechet force-pushed wip/73851-toulouse-2022-activity-catalog from fe4acffe0b to 82c803d424 2023-03-08 15:14:00 +01:00 Compare
Owner

Ajouté les filtres via l'url (de la forme ?age=0,3,4&type=sejour). Les critères indisponibles sont également retirés lorsqu'on filtre : à voir si tu gardes ou pas Nicolas : ça retire la possibilité de filtrer à la fois les séjours et les activités régulières, par exemple.

Ajouté les filtres via l'url (de la forme ?age=0,3,4&type=sejour). Les critères indisponibles sont également retirés lorsqu'on filtre : à voir si tu gardes ou pas Nicolas : ça retire la possibilité de filtrer à la fois les séjours et les activités régulières, par exemple.
nroche changed title from toulouse-2022: add json cell to display maelis catalog (#73851) to WIP: toulouse-2022: add json cell to display maelis catalog (#73851) 2023-03-24 08:44:20 +01:00
Author
Owner

Repassé en WIP parce qu'on utilise plutôt une cellule proto-json en attendant que la présentation du catalogue soit finalisée.
https://portail-parsifal.test.entrouvert.org/manage/pages/39/#open:data_jsoncell-304-general

Repassé en WIP parce qu'on utilise plutôt une cellule proto-json en attendant que la présentation du catalogue soit finalisée. https://portail-parsifal.test.entrouvert.org/manage/pages/39/#open:data_jsoncell-304-general
Owner

Repassé en WIP parce qu'on utilise plutôt une cellule proto-json en attendant que la présentation du catalogue soit finalisée.

Je trouve ça une assez mauvaise idée, ça empêche un suivi et les relectures. (entre autres)

J'avais déjà recréé #76628, j'y transfère des éléments de cette PR, après avoir vérifié le contenu par rapport à ce qui est actuellement dans la cellule propotype en ligne.

> Repassé en WIP parce qu'on utilise plutôt une cellule proto-json en attendant que la présentation du catalogue soit finalisée. Je trouve ça une assez mauvaise idée, ça empêche un suivi et les relectures. (entre autres) J'avais déjà recréé #76628, j'y transfère des éléments de cette PR, après avoir vérifié le contenu par rapport à ce qui est actuellement dans la cellule propotype en ligne.
fpeters closed this pull request 2023-04-15 11:18:26 +02:00
All checks were successful
gitea/publik-base-theme/pipeline/head This commit looks good

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
6 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#142
No description provided.