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

Imposer la sélection d'une ou plusieurs sources dans la recherche #127

Open
4 tasks
vmttn opened this issue Oct 6, 2023 · 0 comments
Open
4 tasks

Imposer la sélection d'une ou plusieurs sources dans la recherche #127

vmttn opened this issue Oct 6, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@vmttn
Copy link
Contributor

vmttn commented Oct 6, 2023

En lien avec l'objectif Qui consomme quoi ?

Actuellement il est possible de faire des recherches sur l'api sans préciser de sources, ce qui retourne des résultats toutes sources confondues.

Il faudrait imposer l'utilisation du paramètre pour miieux identifier ce qui est consommé.

https://github.com/betagouv/data-inclusion/blob/b5f27b3840bd7c384d9c62955e3452445186196a/api/src/data_inclusion/api/entrypoints/fastapi.py#L611-L619

Screenshot de la doc openapi

image

  • checker dans metabase si des consom. font des requêtes sans param sources
  • mettre un cas de test sans le param
  • imposer api/src/data_inclusion/api/entrypoints/fastapi.py
  • réparer les autres tests de la recherche qui n'utiliseraient pas ce param
@vmttn vmttn added good first issue Good for newcomers enhancement New feature or request labels Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant