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

Fleet Management con cambios de DEV014 (CRUD, Swagger etc) #1822

Closed
unjust opened this issue May 15, 2024 · 1 comment
Closed

Fleet Management con cambios de DEV014 (CRUD, Swagger etc) #1822

unjust opened this issue May 15, 2024 · 1 comment

Comments

@unjust
Copy link
Member

unjust commented May 15, 2024

En DEV014 introducimos mas funcionalidades de CRUD y un swagger.

Llevamos estes cambios al proyecto y conversamos un poco sobre la nueva spec de endpoints en Swagger

Hemos conversado un poco sobre el swagger en este issue

@unjust unjust added this to the Fleet Management milestone May 15, 2024
@unjust
Copy link
Member Author

unjust commented May 15, 2024

Los puntos que conversamos de documentacion de API (de otro issue):

de @cros410

Teniendo como base esta documentación creo que hay algo que tenemos que terminar ponernos de acuerdo:

  • Puerto compartido para todas las estudiantes -> Sugerencia el 8080
  • Podemos ponerle el prefijo de /auth para que esté bajo un contexto
  • taxis: Solo dejar esta ruta con los parámetros de page y limit
  • trajectories/{taxiId} Cambiar para que tanto el taxiId, date, page y limit se envíen por query para respetar la jerarquía en la URL. Adicionalmente haría la aclaración del formato de la fecha algo como DD-MM-YYYY
  • trajectories/{taxiId}/export Mismos comentarios que el punto anterior
  • trajectories/latest Nos tenemos que poner de acuerdo sobre las estructuras de respuesta en todos los endpoints. Creo que solo hay una diferencia entre la respuesta de la historia 3 y 4. Sugiero que nos quedemos con la de la historía número 4 donde la placa se incluye dentro del contenido

Con estos cambios creo que podemos armar bien el contenido de las pruebas de postman.

de @davidgranados :

creo que trajectories/{{taxiId}} no tiene mucho sentido... si en el path param va el taxi id, no debería ser taxis/{{taxiId}}/trajectories ? si trajectories es el recurso principal me parece que entonces si deberai ser /trajectories?taxiId={{taxiId}}

de yo:

Hay mucho 401 y 403 si no eres authenticado y no eres admin. Debemos conversar si este requisito es necesario por cada endpoint. Creo podemos relajar eso con algunos endpoints como taxis.

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

Successfully merging a pull request may close this issue.

1 participant