search

Jouchef / Partiokokoustilan-tyokalu

Web app build from this project will be used by scout group Rastipartio to track the usage of their inventory and to upkeep database about how many people use the space and when.
0 stars 0 forks source link

Vertaisarviointi #1

Closed mikaelvilen closed 3 years ago

mikaelvilen commented 3 years ago

Sovellusta oli sujuvaa ja mukava käyttää. Sovellus oli selkeästi toteutettu ja kaikki toteutetut toiminnot toimivat.

Käyttäjätunnuksen luonnissa voisi käyttää UNIQUE -määrettä, jotta ei voisi luoda kahta samaa tunnusta. Päiväkirjamerkintä lomakkeen pystyi lähettämään tyhjänä, tähän voisi tehdä validoinnin html:n puolella. Inventaarion tuotteen lisäämisen lomakkeen pystyi myös lähettämään tyhjänä, josta seurasi internal server error 500.

Koodia oli helppo lukea, selkeästi toteutettu. Päiväkirjamerkinnässä voi lisätä minkä tahansa nimen kirjoittajaksi, olisiko tämä parempi toteuttaa tietokannasta niin, että käyttäjä lisää itselleen käyttäjänimen tunnuksen luonnin yhteydessä, jotta päiväkirjamerkinnän kirjoittaja on aina kirjautunut käyttäjä?

Hyvä sovellus!

Jouchef commented 3 years ago

Kiitos kehittävistä kommenteista!