You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
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
The text was updated successfully, but these errors were encountered: