14.12.2018 klo 22:45.
##Yleistä
-Sovellusta voidaan pitää erinomaisena, eikä siihen yleisesti ottaen ole hirmuisen paljon kommentoitavaa. Tämän vuoksi monet ehdotukset alla voivat olla hieman nippeliin keskittyviä. Jokatapauksessa hienoa työtä!
-Sovellusta testattiin sekä valmiilla admin-käyttäjällä että luomalla useampi käyttäjä. Admin käyttäjällä testattiin skeä kirjanmerkkien että kategorioiden lisäämistä, poistamista ja muokkaamista. Sen lisäksi admin-käyttäjällä testattiin käyttäjätunnuksien poistamista. Molemmilla käyttäjillä kokeiltiin myös sisäänkirjautumisen toimivuutta.
##Dokumentaatio
-Erittäin hyvin tehty dokumentaatio, nähdäkseni dokumentaatio on kokonaisuudessaan kurssin laajuus huomioon ottaen erinomainen.
-Etenkin käyttäjäkuvaukset ja sovelluksen käyttöohjeet oli laadittu erinomaisesti!
##Toiminnallisuudet
-Tällä hetkellä kategoria sama kategoria voi olla sekä "parent" että "child" -kategoria samalle kategorialle. Onko tämä harkittu ominaisuus sovelluksessa? Itselleni ominaisuus tuntui hieman epätarkoituksenmukaiselta ja sekavalta. Lopulta kuitenkin makuasia.
##Ulkoasu
-Tämä on nähdäkseni myös lähinnä makuasia, mutta ulkoasu oli mielestäni ehkä hieman sekava uuden käyttäjän kannalta. Valtaosa sovelluksen tavarasta on selaimen yläosassa horisontaalisesti, mikä voi vähän heikentää käytettävyyttä. Koska toiminnallisuuksia on myös paljon, on kokonaisuus siksi sekavahko.
-Ehkä osa toiminnallisuuksista ja linkeistä voisi sijoittaa omiin "dropdown-menuihinsa" tai jotenkin muuten?
-Sovelluksessa oli esimerkiksi kaksi horisontaalista valikkoa. Voisiko toinen toimivampi ratkaisu olla sijoittaa nämä esimerkiksi vertikaalisesti tai keskemmälle?
##Errorit
-Onnistuin saamaan aikaan errorin poistaessani herokussa olevasta sovelluksesta käyttäjää "Demo". Muita käyttäjiä poistaessani admin-tunnuksilla en kuitenkaan saanut aikaan virheitä. Koodia tutkaillessani en saanut selville, mistä error johtuu, mutta olkoon ainakin tiedoksi.
##Muuta
-Koodi oli mielestäni kirjoitettu erinomaisesti, siinä oli esimerkiksi nähdäkseni vältetty hyvin esimerkiksi toistoa. Tästä hyvänä esimerkkinä toimii tiedosto "macros.html"
14.12.2018 klo 22:45.
##Yleistä
-Sovellusta voidaan pitää erinomaisena, eikä siihen yleisesti ottaen ole hirmuisen paljon kommentoitavaa. Tämän vuoksi monet ehdotukset alla voivat olla hieman nippeliin keskittyviä. Jokatapauksessa hienoa työtä!
-Sovellusta testattiin sekä valmiilla admin-käyttäjällä että luomalla useampi käyttäjä. Admin käyttäjällä testattiin skeä kirjanmerkkien että kategorioiden lisäämistä, poistamista ja muokkaamista. Sen lisäksi admin-käyttäjällä testattiin käyttäjätunnuksien poistamista. Molemmilla käyttäjillä kokeiltiin myös sisäänkirjautumisen toimivuutta.
##Dokumentaatio
-Erittäin hyvin tehty dokumentaatio, nähdäkseni dokumentaatio on kokonaisuudessaan kurssin laajuus huomioon ottaen erinomainen.
-Etenkin käyttäjäkuvaukset ja sovelluksen käyttöohjeet oli laadittu erinomaisesti!
##Toiminnallisuudet
-Tällä hetkellä kategoria sama kategoria voi olla sekä "parent" että "child" -kategoria samalle kategorialle. Onko tämä harkittu ominaisuus sovelluksessa? Itselleni ominaisuus tuntui hieman epätarkoituksenmukaiselta ja sekavalta. Lopulta kuitenkin makuasia.
##Ulkoasu
-Tämä on nähdäkseni myös lähinnä makuasia, mutta ulkoasu oli mielestäni ehkä hieman sekava uuden käyttäjän kannalta. Valtaosa sovelluksen tavarasta on selaimen yläosassa horisontaalisesti, mikä voi vähän heikentää käytettävyyttä. Koska toiminnallisuuksia on myös paljon, on kokonaisuus siksi sekavahko.
-Ehkä osa toiminnallisuuksista ja linkeistä voisi sijoittaa omiin "dropdown-menuihinsa" tai jotenkin muuten?
-Sovelluksessa oli esimerkiksi kaksi horisontaalista valikkoa. Voisiko toinen toimivampi ratkaisu olla sijoittaa nämä esimerkiksi vertikaalisesti tai keskemmälle?
##Errorit
-Onnistuin saamaan aikaan errorin poistaessani herokussa olevasta sovelluksesta käyttäjää "Demo". Muita käyttäjiä poistaessani admin-tunnuksilla en kuitenkaan saanut aikaan virheitä. Koodia tutkaillessani en saanut selville, mistä error johtuu, mutta olkoon ainakin tiedoksi.
##Muuta
-Koodi oli mielestäni kirjoitettu erinomaisesti, siinä oli esimerkiksi nähdäkseni vältetty hyvin esimerkiksi toistoa. Tästä hyvänä esimerkkinä toimii tiedosto "macros.html"