Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

Permet la recherche de structures par le nom #397

Merged
merged 2 commits into from
Oct 7, 2024
Merged

Conversation

ggounot
Copy link
Collaborator

@ggounot ggounot commented Oct 3, 2024

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 devrait l'utiliser avec un nombre de caractères minimum (3 par exemple).

Changements :

  • Modèle Structure : indexation du champ name du modèle Structure
  • StructureViewSet : ajout d'un paramètre search permettant de filtrer les structures sur le nom dans le queryset

@ggounot ggounot requested review from ikarius and jbuget October 3, 2024 18:52
Copy link
Contributor

@ikarius ikarius left a comment

Choose a reason for hiding this comment

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

C'est vraiment très cool de pouvoir faire des optimisations de ce style quand on le peut. Merci !

@ggounot ggounot enabled auto-merge (squash) October 7, 2024 10:04
@jbuget
Copy link
Contributor

jbuget commented Oct 7, 2024

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants