Skip to content

🔍 Ajout du filtre objet dans l'API Rest #1556

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 20 commits into from
Apr 25, 2025
Merged

🔍 Ajout du filtre objet dans l'API Rest #1556

merged 20 commits into from
Apr 25, 2025

Conversation

fabienheureux
Copy link
Member

@fabienheureux fabienheureux commented Apr 23, 2025 •
edited
Loading

Description succincte du problème résolu

Ajout du filtre objet dans l'API Rest

🗺️ contexte: API J'agis

💡 quoi: Ajout du filtre objet

🎯 pourquoi: pour leur donner plus de possibilités de filtrage lors de la consommation de nos données

🤔 comment:

  • embarque Add tests for REST API #1035
  • ajout d'une liste des sous catégories affichées
  • ajout d'un filtre django ninja sur le endpoint acteurs pour filtrer sur la base d'un id de sous catégorie

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

✅ Reste à faire (PR en cours)

  • Tester le nouveau code
  • Mocker les appels à api adresse

@fabienheureux fabienheureux added the enhancement New feature or request label Apr 23, 2025
@fabienheureux fabienheureux mentioned this pull request Apr 23, 2025
Closed
@fabienheureux fabienheureux marked this pull request as ready for review April 23, 2025 16:38
@fabienheureux fabienheureux requested a review from a team as a code owner April 23, 2025 16:38
@fabienheureux fabienheureux requested review from kolok and maxcorbeau and removed request for a team April 23, 2025 16:38
@fabienheureux fabienheureux added the minor Utilisé pour les changelog label Apr 23, 2025
@fabienheureux
Copy link
Member Author

Il me reste deux tâches, mais je suis preneur d'une review à ce stade !

@notion-workspace Notion Workspace
Copy link

[API J’Agis] Ajout paternité et filtre objet

@fabienheureux
Copy link
Member Author

j'ai ajouté les tests + mock

fabienheureux
fabienheureux commented Apr 24, 2025
View reviewed changes
fabienheureux
fabienheureux commented Apr 24, 2025
View reviewed changes
fabienheureux
fabienheureux commented Apr 24, 2025
View reviewed changes
@kolok kolok enabled auto-merge (squash) April 25, 2025 15:54
@kolok kolok disabled auto-merge April 25, 2025 15:54
@kolok kolok assigned kolok and fabienheureux and unassigned kolok Apr 25, 2025
@fabienheureux fabienheureux enabled auto-merge (squash) April 25, 2025 16:07
@fabienheureux fabienheureux merged commit 6c21c54 into main Apr 25, 2025
8 of 11 checks passed
@fabienheureux fabienheureux deleted the add-object-filter-rest-api branch April 25, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kolok kolok kolok approved these changes

@maxcorbeau maxcorbeau Awaiting requested review from maxcorbeau maxcorbeau is a code owner automatically assigned from incubateur-ademe/lvao-admin

Assignees

@fabienheureux fabienheureux

Labels
enhancement New feature or request minor Utilisé pour les changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fabienheureux @kolok