feat: harmonize the "list" API endpoints - Githubissues

betagouv / recommandations-collaboratives

Outil support Recommandations-Collaboratives basé sur le framework django

https://recommandations-collaboratives.beta.gouv.fr

GNU Affero General Public License v3.0

6 stars 9 forks source link

feat: harmonize the "list" API endpoints #517

Open etchegom opened 4 days ago

etchegom commented 4 days ago

Il s'agit de :

[ ] Une correction de bug
[ ] Une nouvelle fonctionnalité programmée
[x] Une nouvelle fonctionnalité spontanée

Décrivez vos changements

tous les endpoitns d'API retournent un objet et non pas une liste
la pagiantion est active par défaut (activé dans les settings classiques DRF)
avoir à dispo le paramètre "count"
simplification des vues d'API à l'aide de ListAPIView
regrouper tous les appels d'API dans api.js

Checklist d'acceptation de revue de code

[ ] Aucun texte ne fait référence à du vocabulaire spécifique d'un métier
[ ] Mon code est auto-documenté ou la documentation a été mise à jour/créée
[ ] La gestion du multi-portail a été prise en compte
[ ] L'accessibilité a été prise en compte
[ ] Pre-commit est configuré et a été lancé
[ ] Des tests couvrant le code changé ont été ajoutés/modifiés
[ ] L'ensemble des tests front et back sont au vert

Demandes

[ ] Je souhaite un déploiement en préproduction

etchegom commented 3 days ago

:wave: @Jeremy-Bojko, ce serait possible de passer les tests e2e sur cette branche stp?

Jeremy-Bojko commented 3 days ago

👋 @Jeremy-Bojko, ce serait possible de passer les tests e2e sur cette branche stp?

Je te fais ça sur la pause dej

glibersat commented 1 day ago

@Jeremy-Bojko Qu'est ce que ça a donné ? est-ce que c'est ok sur la compat arrière ?

etchegom commented 1 day ago

Les tests échouent pour le moment Je vais refaire une passe