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 - PR URL template si application publiée #254

Open
lecault opened this issue Jun 21, 2023 · 5 comments
Open

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

lecault opened this issue Jun 21, 2023 · 5 comments

Comments

@lecault
Copy link
Collaborator

lecault commented Jun 21, 2023

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

Scénario :

  • Je crée une application avec un template
  • Je publie l'application
  • Je clique sur télécharger votre application
  • J'ouvre le XML : le lien vers le template n'est pas bon. Il renvoie vers la version brouillon.

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
Copy link
Member

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

@lecault
Copy link
Collaborator Author

lecault commented Jun 22, 2023

Tu m'as mis le doute mais dans l'issue #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
Copy link
Member

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):

#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
Copy link
Collaborator

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
Copy link
Member

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 :

  • une contrainte de nommage unique par application.
  • aucun fichiers utile au studio pour le versionnement, les preview etc.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants