Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MEG - Finaliser la fonction d’import de thématiques Mviewer existantes #159

Closed
Gaetanbrl opened this issue Feb 3, 2023 · 25 comments
Closed
Milestone

Comments

@Gaetanbrl
Copy link
Member

Gaetanbrl commented Feb 3, 2023

Issue développée dans le cadre d'une prestation JDev / Megalis Bretagne / GeoBretagne

Histoire

En tant que utilisateur,
Je souhaite pouvoir visualiser et filtrer et importer des thématiques,
Pour afin d’améliorer la fenêtre d’import de thématiques existantes

Description complémentaire

Le principe de cette fonctionnalité est de proposer à l’utilisateur des thématiques d’applications mviewer « prêtes à l’emploi ». Les thématiques sont des thématiques d’application mviewer publiées sur le Web et listées dans un fichier de type csv. Une amorce d’implémentation existe déjà dans mviewer studio à partir de l’inventaire suivant : https://geobretagne.fr/minicatalog/csv
image

La source de donnée associée se trouve ici : https://lite.framacalc.org/codesprintbegmeil

L’import de thématiques fonctionne actuellement avec tous les types de couches y compris les customlayers dès lors que toutes les URL des ressources nécessaires sont précisées en absolu et servies avec les entêtes CORS.

Il est nécessaire de revoir l’écran permettant de sélectionner la /les thématiques avec les ajouts suivants afin de faciliter l'exploration du catalogue de thèmes et la compréhension de chaque thème :

  • Filtrage des thématiques par nom ou mot clé, producteur, statut (observations = ‘Ok’) ;
  • Affichage de la description, du logo, observations de la source (Ok, pas à jour...) ;
  • Bouton sélectionner plus accessible ;
  • Créer une option d’affichage pour :
    • afficher toutes les couches de la thématique au démarrage ;
    • aucune couche au démarrage ;
    • affichage des couches par tel que défini dans le XML source.
@Agath21
Copy link
Collaborator

Agath21 commented Mar 22, 2023

Proposition d'interface et parcours utilisateur

Dans le module "Thématiques & Données", je clique sur "Importer", un nouvelle modale s'ouvre avec les thématiques externes :

Studio2_importExternThem

Pour chaque thématique, je visualise :

  • Le nom
  • La description
  • Le statut
  • Le créateur

J'ai la possibilité de les filtrer via une barre de recherche en haut à droite.

Pour améliorer la visibilité, nous proposons d'ajouter un système de pagination.

@lecault
Copy link
Collaborator

lecault commented Mar 24, 2023

N'afficher que les thématiques dont le statut est à jour (ne pas afficher les obsolètes).
Ok pour la pagination
Recherche sur le titre, la description, le créateur et les mots clés (subject).
Ok pour ne pas afficher le logo

@spelhate
Copy link
Collaborator

Ok de mon côté également.

@Agath21
Copy link
Collaborator

Agath21 commented Apr 3, 2023

Développement à réaliser :

  • Mettre à jour l'interface en affichant le nom, la description, l'organisme
  • Barre de recherche pour filtrer les thématiques
  • Pagination et gestion de l'affichage
  • Filtrer les thématiques et n'afficher que celle avec le statut "à jour"
  • Créer un nouveau paramètre pour la gestion de l'affichage des couches au sein de la thématique
  • Documentation

@Agath21
Copy link
Collaborator

Agath21 commented Apr 4, 2023

Créer un nouveau paramètre pour la gestion de l'affichage des couches au sein de la thématique

Pour gérer l'affichage des couches au sein d'une thématique externe lors du démarrage de l'application, création d'un nouveau paramètre layersvisibility spécifique aux thématiques externes avec les valeurs possibles :

  • default : Afficher les données par défaut comme défini dans l'application source
  • all : Afficher toutes les données de la thématique externe
  • none : Aucune donnée n'est affichée

Exemple :

<theme id="education" url="https://kartenn.region-bretagne.fr/territoires/apps/default.xml" name="Education et formation" collapsed="true" icon="false" layersvisibility="none"> 
</theme>

@Gaetanbrl
Copy link
Member Author

création d'un nouveau paramètre layersvisibility spécifique aux thématiques externes avec les valeurs possibles

@lecault @spelhate est-ce que ce nouveau paramètre studio vous convient ?

@lecault
Copy link
Collaborator

lecault commented Apr 4, 2023

ça correspond au besoin oui merci.

@spelhate
Copy link
Collaborator

spelhate commented Apr 6, 2023

Ok pour moi. Issue mise à jour côté mviewer : mviewer/mviewer#616

@lecault
Copy link
Collaborator

lecault commented Apr 6, 2023

Dans cette issue, il y avait aussi la possibilité de modifier le nom de la thématique (le name).
Le dev est côté mviewer, côté studio, prévoir de modifier le nom de la thématique.

Gaetanbrl added a commit to jdev-org/mviewerstudio that referenced this issue Apr 12, 2023
Finaliser la fonction d'import des thématiques mviewer#159
@lecault
Copy link
Collaborator

lecault commented Apr 17, 2023

Suite à test sur la branche-159

On n'a pas pu testé l'import dans mviewer à cause problème de proxy

@Gaetanbrl
Copy link
Member Author

On n'a pas pu testé l'import dans mviewer à cause problème de proxy

Le problème n'est pas réglé avec la whitelist du proxy backend studio ?

@spelhate
Copy link
Collaborator

Ok pour moi. Issue mise à jour côté mviewer : mviewer/mviewer#616

PR mviewer réalisée ce jour : mviewer/mviewer#753

@lecault
Copy link
Collaborator

lecault commented Apr 24, 2023

Un truc que je n'avais pas fait attention (c'est mineur).

Lorsqu'on importe une thématique dans le XML studio renseigne toujours les valeurs suivantes pour les paramètres collapsed et icon :
collapsed="true" icon="false"

Ce n'est pas très propre car ces 2 valeurs ne sont pas prises en compte dans mviewer.
Je propose que studio ne génère aucune de ces valeurs.

<theme id="education" url="https://kartenn.region-bretagne.fr/territoires/apps/default.xml" name="Education et formation" collapsed="true" icon="false"> 

devient

<theme id="education" url="https://kartenn.region-bretagne.fr/territoires/apps/default.xml" name="Education et formation"> 

J'ai testé sur mviewer ça ne pose pas de soucis.

@Gaetanbrl
Copy link
Member Author

Gaetanbrl commented Apr 25, 2023

Lorsqu'on importe une thématique dans le XML studio renseigne toujours les valeurs suivantes pour les paramètres collapsed et icon :
collapsed="true" icon="false"

Ca donne quoi en v3.2 (latest) @lecault ? Tu le reproduis ?

@lecault
Copy link
Collaborator

lecault commented Apr 25, 2023

Lorsqu'on importe une thématique dans le XML studio renseigne toujours les valeurs suivantes pour les paramètres collapsed et icon :
collapsed="true" icon="false"

Ca donne quoi en v3.2 (latest) @lecault ? Tu le reproduis ?

OUI

@Gaetanbrl
Copy link
Member Author

ok du coup c'est une issue hors MEG => A traiter si on a le temps dans le cadre des travaux MEG donc.

@lecault
Copy link
Collaborator

lecault commented May 3, 2023

J'ai fait une issue spécifique #196

@Gaetanbrl
Copy link
Member Author

@lecault @spelhate avez-vous pu tester ?

Pour rappel le proxy est utile sur notre instance pour utiliser vos flux avec le studio (voir ici également)

  • proxy activé

On peut appeler les flux de la région.
On ne peut pas utiliser les thématiques externes car mviewer utiliser la valeur du proxy.

  • proxy désactivé

On ne peut pas utiliser les flux région (erreur CORS).
Mais on peut utiliser les thématiques externens.

@Agath21
Copy link
Collaborator

Agath21 commented May 9, 2023

Pour information, nous avons désactivés le proxy sur https://gis.jdev.fr/mviewerstudio/ pour que vous puissiez tester l'import des thématiques externes et la gestion de l'affichage des couches.

@spelhate
Copy link
Collaborator

spelhate commented May 10, 2023

Pas pu tester car pas de proxy sur https://gis.jdev.fr/mviewerstudio/ . j'ai une fenêtre vide lorsque je souhaite importer une thématique :
image

Blocage d’une requête multiorigine (Cross-Origin Request) : la politique « Same Origin » ne permet pas de consulter la ressource distante située sur https://geobretagne.fr/minicatalog/csv. Raison : l’en-tête CORS « Access-Control-Allow-Origin » est manquant.

@Gaetanbrl
Copy link
Member Author

Etrange on ne reproduis pas @Agath21 et moi :

image

@lecault
Copy link
Collaborator

lecault commented May 10, 2023

Je vois les thématiques aussi

@spelhate
Copy link
Collaborator

Oui ça marche maintenant chez moi également.

@Gaetanbrl
Copy link
Member Author

A fermer @spelhate @lecault ?

@spelhate
Copy link
Collaborator

Ok.

@Gaetanbrl Gaetanbrl added this to the MEGALIS - JDEV milestone Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants