Vertaisarviointi 2

[tirhelen]

• Sovellukseen suunnitellut toiminnallisuudet vaikuttavat pääosin toimivan hyvin. Arvostelun lähettämisen jälkeen kuitenkin tuli "404 Not found The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again." virheilmoitus. Arvostelu tuli kuitenkin näkyviin sivulle.
• Sovelluksessa on "Email the owner"-mahdollisuus, mutta missään ei ole näkyvissä omistajan sähköpostiosoitetta.
• Pystyin tekemään varauksen vuoteen 2290 saakka, joten voisi olla ehkä hyvä antaa varauksille jokin maksimipituus ja tarkistaa se ennen kuin varaus on hyväksytty.
• Ilmeisesti myöskään mökin nimen/kuvauksen pituutta ei tarkisteta, sillä sivulla näkyy (ainakin tällä hetkellä kun tätä kirjoitan) mökki, jonka nimi on monta kertaa "testi".
• Tämä ei ollut suunnitelmassa, mutta jos sovelluksesta haluaisi vielä enemmän "oikeaan elämään" sopivamman, olisi hyödyllinen lisä, jos tavallinen käyttäjä pystyisi perumaan tekemiään varauksia.
• Sovelluksen ulkoasu on yksinkertaisen siisti. Sitä on helppo käyttää ja sivujen välillä pystyy navigoimaan edestakaisin vaivattomasti.
• Koodi on siistiä ja selkeää. Se on erittäin hyvin jaoteltu omiin kansioihin, tiedostoihin, funktioihin ja luokkiin. Dokumentaatio on myös riittävän täsmällinen niin, että sovelluksen halutusta toiminnasta saa selkeän käsityksen.
• Kokonaisuudessaan hyvää työtä!! Tsemppiä kurssin loppuun :)

[mjjs]

• Arvostelun jälkeinen uudelleenohjaus on näköjään mennyt refaktoroitaessa rikki. Hyvä huomio! (Tämän takia pitäisi aina kirjoittaa testit)
• Email the owner on mailto: -tyyppinen linkki, joka avaa sähköpostisovelluksen ja käyttää lähetykseen oikeaa osoitetta. Vaihdoin kuitenkin tekstin nyt hieman käyttäjälle ystävällisempään muotoon.
• Nuo käyttäjän syötteen pituuksien validoinnit on hyvä pointti. Pitää lisäillä ne.
• Oman varauksen poistamiseen/perumiseen olen itseasiassa toteuttanut back-end koodin, mutta minulta puuttuu vielä nappi mistä se tehdään.

Kiitos huomioista!