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

Delivery of 5.1#3

Open
macheitor wants to merge 108 commits into
mainfrom
5.1
Open

Delivery of 5.1#3
macheitor wants to merge 108 commits into
mainfrom
5.1

Conversation

@macheitor

Copy link
Copy Markdown
Owner

No description provided.

@macheitor
macheitor temporarily deployed to retrochatmachado May 30, 2022 10:36 Inactive
@StratocasterO

Copy link
Copy Markdown

Molt bona feina, Víctor! Ahí van els comentaris:

  • Sobre l'estructura del back, ben feta, però es podria simplificar una mcia més el app.js separant les rutes i demés (el sistema de fer una classe server no és mala idea 😉)
  • De funcionalitats generals, només hauries de corregir lo de que puguis iniciar sessió en dues finestres i passin coses lletges
  • Sobre documentació, es podria afegir el que us comentava: descripció i instruccions generals, tecnologies usades, descripció dels endpoints i dels esdeveniments de sockets, etc. Afegir captures al README.me també el fa més guai per quan el vegi algú que entri al teu Github
  • Tot i que no aplica per avaluar, l'estructura del frontend és una mica dispersa, mira d'ordenar una mica els .js perquè tingui sentit (i ho pots posar tot en un server que serveixi el index.js)

Has fet un molt bon exercici, els següents passos per tu serien arreglar el responsive perque funcioni a mòbil i intentar replicar-ho amb algun framework rollo React! Well done, company 💪🏻

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