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

[BACKEND/API/FRONTEND] Uniformisation des endpoints API (Domain/Paramètres/Pagination) #411

Open
rv2931 opened this issue Jan 26, 2025 · 1 comment

Comments

@rv2931
Copy link
Collaborator

rv2931 commented Jan 26, 2025

Je pense qu'il faudrait revoir un peu les différents endpoints API pour coller au mieux aux Domains pensés initialement
De plus une pagination a été mise en place (norme PostgREST - header (range/content-range)) mais pour le mooment seuls les endpoints qui ont pu poser problème de perfo pendant la saison ont été paginés.
Une amélioration/mise au propre serait d'uniformiser et généraliser cette pagination sur les différents endpoints. Cela risque d'avoir quelques impacts par contre sur les format des réponses API et donc aussi côté frontend

Il y a la gestion du cache aussi à mettre au propre et uniformiser

@rv2931
Copy link
Collaborator Author

rv2931 commented Feb 8, 2025

Pour cette partie, idéalement et si on garde le principe de l'architecture Domain Driven Design qui a été inité au départ du projet, il faudrait surtout repenser les endpoints pour coller au mieux aux activités métiers
Actuellement les endpoints sont pensés pour colle eux repositories, les repositries eux-même pensés pour coller à la structure de la BDD. Les endpoints sont sûrement à repenser/réorganiser sur une vision métier
J'ai pas forcément d'exemple et y a sûrement des endpoints qui correspondent déjà à cette vision
En plus de ça il y a toujours l'aspect technique à savoir le format uniformiser le format des réponses, et uniformiser la gestion de la pagination et éventuellement du filtrage si cela est possible

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

No branches or pull requests

2 participants