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

Chargement de la liste des application si config non chargée #202

Open
Gaetanbrl opened this issue May 11, 2023 · 0 comments
Open

Chargement de la liste des application si config non chargée #202

Gaetanbrl opened this issue May 11, 2023 · 0 comments
Labels
bug mineur not obstructive bug

Comments

@Gaetanbrl
Copy link
Contributor

Gaetanbrl commented May 11, 2023

Description

Lorsque du chargement initial, l'utilisateur peut cliquer sur "Ouvrir un projet existant" de manière très rapide.

L'ouverture de la modal entraîne alors un appel de la liste des applications selon une URL récupérable dans la config :

$("#mod-importfile").on('shown.bs.modal', function () {

SI la config (config.json) n'est pas encore disponible, l'ouverture de la modal provoque une erreur :

.js:1531 Uncaught TypeError: Cannot read properties of undefined (reading 'api')
    at Object.getListeApplications (mv.js:1531:33)
    at HTMLDivElement.<anonymous> (mviewerstudio.js:1160:8)
 

Ne voyant aucun mécanisme d'événement sur le chargement de la config, je pense que ce problème est déjà présent en version latest.

Un message invitant l'utilisateur à recharger la page serait une alternative peu coûteuse.
Autrement, rajouter un event / listener avec un loader pour montrer que la liste est en cours de chargement, tant qu'on a pas récupéré la config + la réponse de l'appel fetch.

@Gaetanbrl Gaetanbrl added the bug label May 11, 2023
@Gaetanbrl Gaetanbrl changed the title MEG - Chargement de la liste des application si config non chargée Chargement de la liste des application si config non chargée May 16, 2023
@Gaetanbrl Gaetanbrl added the mineur not obstructive bug label May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug mineur not obstructive bug
Projects
None yet
Development

No branches or pull requests

1 participant