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

[BO -Tableau de bord] Problème chargement du tableau de bord #3109

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Sep 30, 2024

Ticket

#3103
image

Description

Augmentation du timeout pour les requêtes du dashboard dont une peut mettre plus de 15 secondes pour le super-admin

Correction d'un problème de chargement de widget entraînant des widgets non chargés correctement dans le tableau de bord ou initKPI peut répondre avant initSettings

image

Les modifications apportées garantissent que les réponses des services réponde dans l'ordre voulu

Changements apportés

  • Surcharge des fonction callback initSettings et initKPI par des promesses initSettingsWithPromise et initKPIWithPromise
  • Gestion des promesses lors de l'appel via async/await pour garantir l'ordre des réponses.
  • Augmentation du timeout de 15 à 30s
  • Mise à jour de la config eslint

Pré-requis

Charger la base de prod

make load-data

Builder les assets

make npm-watch

Tests

  • Se connecter en super admin et vérifier que le tableau de bord affichent bien tous les widget et sans erreur
  • Refaire un appel, vérifier que le tableau de bord profite du cache et que tous les widgets s'affichent sans erreur
  • Tester en tant que RT et agent

Copy link

sonarcloud bot commented Sep 30, 2024

@sfinx13 sfinx13 marked this pull request as ready for review September 30, 2024 12:09
@sfinx13 sfinx13 changed the title [BO -Tableau de bord] Problème chargement de bord [BO -Tableau de bord] Problème chargement du tableau de bord Sep 30, 2024
Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relecture OK et tests OK en SA

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

Successfully merging this pull request may close these issues.

2 participants