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

Recherche de structures par le nom pour les admins et managers #21

Merged
merged 13 commits into from
Oct 29, 2024

Conversation

ggounot
Copy link
Contributor

@ggounot ggounot commented Oct 23, 2024

ℹ️ PR créée à partir de la PR originale gip-inclusion/dora-front#458 suite à la création du monorepo.

Dépendance back : gip-inclusion/dora-back#397

Problème : la création d'un service depuis un modèle cause un appel au point d'API /structures/ qui liste toutes les structures de la base lorsqu'on est admin. La taille de la réponse atteint 5 Mo et provoque un gros ralentissement voire un plantage de la page.

Solution : ajout d'un paramètre de recherche textuel sur le nom de la structure pour limiter le nombre de résultats. Le front l'utilise avec 3 caractères minimum.

Changements :

  • getManagedStructure() : ajout d'un paramètre de recherche textuel et utilisation du paramètre search du endpoint ;
  • FieldsStructure : utilisation d'une fonction de recherche ;
  • Page de création de service depuis un modèle : création d'un mode de recherche de structure lorsque l'utilisateur est admin ou gestionnaire au lieu de récupérer toutes les structures.

@ggounot ggounot requested review from ikarius and jbuget October 23, 2024 14:32
@ggounot
Copy link
Contributor Author

ggounot commented Oct 23, 2024

@ggounot ggounot self-assigned this Oct 24, 2024
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

Déjà validé, avec quelques nitpicks (d'où le besoin de revalider)

LGTM 🚀

@jbuget jbuget added the feature New feature or request label Oct 28, 2024
@ggounot ggounot merged commit cae9e1f into main Oct 29, 2024
7 checks passed
@ggounot ggounot deleted the recherche-structures branch October 29, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants