search

mviewer / mviewerstudio

mviewer generator
GNU General Public License v3.0
12 stars 26 forks source link

MEG - PR URL template si application publiée #254

Open lecault opened 1 year ago

lecault commented 1 year ago

Reproduit à GéoBretagne et Mégalis en mode non connecté. Lié PR #205

Scénario :

Exemple d'URL : https://geobretagne.fr/mviewer/apps/store/megalis_bretagne/1ba214f0179f/test/templates/lycee.mst

URL attendue : https://geobretagne.fr/mviewer/apps/prod/megalis_bretagne/test/templates/lycee.mst

Gaetanbrl commented 1 year ago

Il me semblait qu'on souhaité que le téléchargement réalisé un ZIP de la dernière version (donc brouillon).

lecault commented 1 year ago

Tu m'as mis le doute mais dans l'issue https://github.com/mviewer/mviewerstudio/issues/226 on parle bien de 2 urls : brouillon et en ligne. L'idée est surtout qu'un utilisateur ai la possibilité de télécharger un XML avec des urls propres.

Gaetanbrl commented 1 year ago

J'ouvre le XML : le lien vers le template n'est pas bon. Il renvoie vers la version brouillon. L'idée est surtout qu'un utilisateur ai la possibilité de télécharger un XML avec des urls propres.

Je comprend donc qu'une URL de brouillon n'est pas propre ?

L'issue de référence est la suivante (testée / validée):

https://github.com/mviewer/mviewerstudio/issues/234

De mémoire, on avait vu avec @spelhate que la dernière version était la version à télécharger. Cette fonctionnalité avait par ailleurs donné l'idée d'avoir la possibilité de télécharger une version spécifique d'une application (via le menu des versions).

Il faudrait donc pouvoir avoir le choix de télécharger la version publiée OU la dernière version. Ce point est une évolution.

spelhate commented 1 year ago

De mémoire, on avait vu avec @spelhate que la dernière version était la version à télécharger. Cette fonctionnalité avait par ailleurs donné l'idée d'avoir la possibilité de télécharger une version spécifique d'une application (via le menu des versions).

ça ok. Mais la logique est d'avoir dans le fichier apps/store/megalis/mon_appli.xml une url vers le template : apps/store/megalis/mon_appli/templates/mon_template.mst et pas un truc du genre apps/store/megalis_bretagne/1ba214f0179f/mon_appli/templates/lycee.mst. Il semble logique qu'en téléchargeant une application, le xml pointe sur le template qui est dans le dossier templates.

Gaetanbrl commented 1 year ago

L'URL suivante :

apps/store/megalis_bretagne/1ba214f0179f/mon_appli/templates/lycee.mst

... pointe bien vers le répertoire des templates pour l'application de brouillon, je ne saisi pas pourquoi ce n'est pas bon vu que la dernière version c'est le brouillon, je n'ai pas d'autre URL.

Sur le fonctionnement :

Pour le brouillon, on a un ID unique par application qui est le nom du répertoire qui contient toutes les ressource pour l'application de brouillon.

Dans ce répertoire nommé avec un ID unique, on a d'autres ressources qui ne sont pas que le XML et le répertoire contenant les templates.

On trouvera notamment un répertoire temporaire utilisé pour générer le ZIP, un répertoire de preview, les fichiers GIT etc. Je remet ca dans l'issue.

Il n'y a que lors de la publication qu'on a une URL type

apps/public/megalis/mon_appli.xml

... car dans le répertoire de publication, nous n'avons :

Si vous désirez une URL "propre" ce sera l'URL de publication, ce qui n'est pas la dernière version.