Skip to content
This repository was archived by the owner on Jul 6, 2023. It is now read-only.

Delivery of 4.1 #1

Open
macheitor wants to merge 7 commits into
mainfrom
4.1
Open

Delivery of 4.1 #1
macheitor wants to merge 7 commits into
mainfrom
4.1

Conversation

@macheitor

Copy link
Copy Markdown
Owner

NOTA: No he sabido como indicarle a las macros del Postman la ruta relativa de las imágenes de la carpeta img.

@StratocasterO

Copy link
Copy Markdown

Bona feina, Víctor! Alguns comentaris:

  • Molt ben fet amb l'estructura del server (no entenc la carpeta startup, però... Podria estar també a routes, no?)
  • La ruta /upload fa tot i controla molt bé tots els errors... Excepte que no puja les fotos al server! 😂
  • La ruta /time està correcte però el header de CORS no s'està activant, sembla que hi ha algun error tonto
  • Molt ben fet també controlant tots els possibles errors i les rutes inexistents

Em penso que no em deixo res! Arregla aquests detalls per la màxima nota 😉

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants