search

JohnnyLAmpAz / smartmag

Repo del progetto per il corso di Ingegneria del Software
4 stars 0 forks source link

Merge del branch DEV nel MAIN #11

Closed JohnnyLAmpAz closed 8 months ago

JohnnyLAmpAz commented 8 months ago

Con questa PR proporrei di approvare quanto abbiamo fatto fino ad ora mergiando nel main e di eliminare il branch GeneralDevelopment. Così, invece che unire i nostri personali in GeneralDevelopment e solo successivamente nel main, gestiamo il tutto con delle pull request direttamente sul main.

Cosa ne dite?

DavideBrambilla98 commented 8 months ago

Per me va bene fare il merge. La classe OrderModel è già stata aggiornata sulla base della review alla PR #9 La classe BoxModel mi sembra ok. Mancano le modifiche richieste nella review alla PR #10 ma si possono effettuare anche dopo il merge volendo dato che non sono invasive.

MassimilianoGervasoni commented 8 months ago

sto apportando ora le modifiche alla classe BoxModel, ho quasi finito devo solo creare il metodo che restituisce le istanze dal database con la treeMap; per il resto sono d'accordo