Skip to content

fix(@multi-frontend/shared): accueil vide au premier lancement#77

Closed
rdupret wants to merge 2 commits intounivlorraine:developfrom
rdupret:fix-empty-home
Closed

fix(@multi-frontend/shared): accueil vide au premier lancement#77
rdupret wants to merge 2 commits intounivlorraine:developfrom
rdupret:fix-empty-home

Conversation

@rdupret
Copy link
Contributor

@rdupret rdupret commented Oct 1, 2024

Bonjour,

Voici un PR pour corriger un problème d'affichage de l'écran d'accueil lors du tout premier lancement de l'application dans certains cas.

Bonne fin de journée


Checklist de PR

Veuillez vérifier que votre PR respecte bien les indications suivantes :

  • Votre PR pointe vers la branche develop
  • Votre PR suit les différentes étapes du guide de contribution : https://www.esup-portail.org/wiki/x/KQCeUQ
  • Les modifications ont été testées de votre côté, cela implique également des tests sur des périphériques (iOS + Android) si le client a évolué
  • La documentation a été mise à jour et prend en compte les changements (fichiers README, Wiki Esup)

Type de PR

Quel type de changement concerne cette PR ?

  • Bug Fix
  • Nouvelle Feature
  • Mise à jour de la documentation (README, CHANGELOG, CONTRIBUTING)
  • Style (SCSS, Assets)
  • Refactoring de code
  • Ajout de tests
  • Build (scripts npm, .sh)
  • CI
  • Chore (nouvelle Release, maj de dépendances)
  • Revert

Cette PR implique un Breaking Change ?

  • Oui
  • Non

rdupret and others added 2 commits October 1, 2024 16:01
Tous les observables combinés émettent bien un résultat mais lors du
tout premier lancement, bien que le store Elf soit initialisé, les
données ne sont pas encore écrites dedans et un tableau vide est renvoyé
au niveau des features, ce qui a pour effet d'afficher une page blanche.
@benjhoo
Copy link
Collaborator

benjhoo commented Oct 9, 2024

Le problème n'apparait plus avec la correction de la PR #82

@benjhoo benjhoo closed this Oct 9, 2024
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.

3 participants