Délai intempestif lié à publishedWebsiteURL

[maiwann]

Suite à un test utilisateur, l'atelier est lent (et cela engendre des misclick) : 10s pour charger les paramètres

C'était pendant un test utilisateur pour les mini-sites Framalibre, et cela a été lent :

• Pour accéder à l'atelier (liste des pages)
• Pour ouvrir une page (la seule existante)
• Pour aller sur la page des Paramètres

La personne a la fibre, donc ça ne doit pas être sa connexion qui explique la lenteur. Dans le reste du parcours, tout se déroulait bien.

[maiwann]

Je crée cette issue pour mémoire, si on nous remonte ce problème plusieurs fois je la prioriserai plus haut.

[maiwann]

Aha je reproduis le problème quand je me connecte avec Github 🕵️ et mes 2 testeureuses utilisaient Github aussi.

Avouez c'est une feature :p

[DavidBruant]

Il est possible que ça soit une lenteur de l'API github (ils font ce qu'ils peuvent pour que ça soit rapide, mais ce n'est pas une science exacte et encore moins fiable)

Avec @Ynote , on a dans l'idée de rajouter des loader, parce que c'est chiant de ne pas avoir de retours visuel quand c'est lent
Maiana l'a beaucoup demandé aussi

[maiwann]

Je trouve que c'est beaucoup plus lent qu'auparavant, et systématique, donc ça me donnait l'impression que c'était une nouveauté :x

[hannaeko]

(un⋅e des testeureuses en question o/)

J'ai fait un peu de debug pour tenter de savoir ce qui se passait à la suite du test et j'ai trouvé ça, je sais pas si ça peut aider.

• Certaines pages font appel à setCurrentRepositoryFromQuerystring
• Cette fonction fait appel à setBaseUrlInConfigIfNecessary
• Dans le cas où l'origin est Github, ça récupère currentRepository.publishedWebsiteURL, https://github.com/Scribouilli/scribouilli/blob/principale/assets/scripts/actions/current-repository.js#L145-L150
• currentRepository est instancié, un peu avant l'appel à setBaseUrlInConfigIfNecessary, https://github.com/Scribouilli/scribouilli/blob/principale/assets/scripts/actions/current-repository.js#L89-L104
• publishedWebsiteURL est set après un délai de 10 secondes dans le constructeur de ScribouilliGitRepo, https://github.com/Scribouilli/scribouilli/blob/principale/assets/scripts/scribouilliGitRepo.js#L30-L41

J'aurais tendance à penser que les 10 secondes d'attente correspondent au 10 secondes du setInterval. (J'aurais bien tenté de vérifier ça mais j'ai pas eu la foi de setup l'environnement de développement)

[DavidBruant]

ah oui, bien vu !
Ça doit être lié !

[DavidBruant]

J'ai changé le délai à une seconde

6dc9ef7

C'est une mesure temporaire
J'aimerais prendre le temps de creuser un jour pour trouver une solution plus viable

[maiwann]

Du coup d'ici là on clôt cette issue ? on en ouvre une autre ? Ou on la garde le temps de faire la solution plus viable ?

[DavidBruant]

On la garde ouverte