Sovelluksen README näyttää oikein perusteelliselta. Sovelluksen idea ja toiminnot sekä tietokantarakenne on kuvattu selkeästi ja ymmärrettävästi. Hyvä, että jätit välipalautus 2:n kommentit näkyviin, sillä näen siitä, että sovellus on edistynyt sormivammasta huolimatta (pikaista paranemista!).
Testausohjeista: pääsin testaamaan sovellusta pienellä säätämisellä. Ehdotan, että väännät rautalangasta ja lisäät ohjeeseen virtuaaliympäristön aktivointikäskyt:
python3 -m venv venv
source venv/bin/activate
Ne ovat tosiaan tämän kurssin osallistujille itsestäänselviä, mutta ulkopuolisille käyttäjille voivat olla tarpeen – oletan, että jätät valmiin sovelluksen Githubiin julkiseksi ja lisäät sen portfolioosi.
En myöskään keksinyt, miten saan käynnistettyä sovelluksen ajalalla routes.py-tiedoston koodin, kuten ohjeessa neuvoit, joten lisäsin app.py -tiedoston loppuun rivin import routes, jonka jälkeen flash run -käskyllä terminaalissa pääsin testaamaan sovellusta selaimessa.
Loin käyttäjätilin, kirjauduin ulos ja yritin sitten sisään väärällä salasanalla – sovellus antoi asianmukaisen virheilmoituksen. Kokeilin myös luoda käyttäjätilin niin, että jätin lomakkeessa kaikki kentät tyhjiksi. Pop up -ilmoitus vakuutti, että tili on luotu, mutta niillä ei pystynyt kirjautumaan sisälle. Jos haluaa viilata pilkkua, tässä kohtaa voisi ilmoitusten suhteen ottaa huomioon, mitä kaikkea käyttäjä voi yrittää, mutta sovellus ei kuitenkaan antanut kirjautua tyhjillä tunnuksilla mikä on se pääasia.
Sitten loin ohjeita noudattaen admin-tunnuksen, lisäsin kauppaan onnistuneesti pari tuotetta ja palasin peruskäyttäjän tunnuksella katsomaan niitä. Testasin muutkin sovelluksessa olevat toiminnallisuudet (tuotteiden arvioinnin, arviointien lukemisen ja tuotteen etsimisen haulla, sekä adminilla tuotteiden poistamisen) ja kaikki tuntuvat toimivan asianmukaisesti. Sessiot toimivat (kirjautumisen jälkeen pääsivulla näkyy käyttäjän nimi) ja sovelluksen ulkoasuakin on näköjään jo vähän säädetty. Keksi vielä nettikaupalle nimi ja lisää se vaikka kuvalogona etu- ja pääsivulle.
Kävin myös lukemassa kooditiedostoja ja kaikki näyttää asianmukaiselta. Koodi on jäsennetty selkeästi ja metodeilla on kuvaavat nimet. Jonkin verran siellä oli myös selventäviä kommentteja, hyvä! tables.sql-tiedostossa oli asianmukaisesti määritellyt tietokantataulut ja sarakkeiden nimet olivat kuvaavat. Tässäkään ei mitään huomautettavaa.
Tehtävänannon mukaisesti yritin kovasti keksiä, miten sovellusta voisi parantaa. Toki sieltä puuttuu vielä ominaisuuksia, mutta listasit itsekin toiminnot jotka tulet vielä toteuttamaan. Ja suoraan sanottuna olin helpottunut, ettei kaikilla muillakaan ole sovellus viimeistelyä vaille valmiina, koska omanikin on vielä ihan vaiheessa :). Pop-up -ikkunat ovat ulkoasultaan vähän virheilmomaisia, joten jos voit muokata niiden ulkoasua, se voisi olla yksi säädettävä asia. Mutta tämä on jo ihan pilkun viilaamista ja ei missään nimeessä sovelluksen toiminnallisuuksiin vaikuttava asia.
Sovellus näyttää tässä vaiheessa oikein hyvältä ja toiminnot toimivat niin kuin pitääkin. Kyllä tämä deadlineen mennessä valmistuu. Tsemppiä viimeisille viikoille!
Tartsi
commented
1 year ago
Sovelluksen README näyttää oikein perusteelliselta. Sovelluksen idea ja toiminnot sekä tietokantarakenne on kuvattu selkeästi ja ymmärrettävästi. Hyvä, että jätit välipalautus 2:n kommentit näkyviin, sillä näen siitä, että sovellus on edistynyt sormivammasta huolimatta (pikaista paranemista!).
Testausohjeista: pääsin testaamaan sovellusta pienellä säätämisellä. Ehdotan, että väännät rautalangasta ja lisäät ohjeeseen virtuaaliympäristön aktivointikäskyt:
Ne ovat tosiaan tämän kurssin osallistujille itsestäänselviä, mutta ulkopuolisille käyttäjille voivat olla tarpeen – oletan, että jätät valmiin sovelluksen Githubiin julkiseksi ja lisäät sen portfolioosi.
En myöskään keksinyt, miten saan käynnistettyä sovelluksen ajalalla
routes.py-tiedoston koodin, kuten ohjeessa neuvoit, joten lisäsinapp.py-tiedoston loppuun rivinimport routes, jonka jälkeenflash run-käskyllä terminaalissa pääsin testaamaan sovellusta selaimessa.Loin käyttäjätilin, kirjauduin ulos ja yritin sitten sisään väärällä salasanalla – sovellus antoi asianmukaisen virheilmoituksen. Kokeilin myös luoda käyttäjätilin niin, että jätin lomakkeessa kaikki kentät tyhjiksi. Pop up -ilmoitus vakuutti, että tili on luotu, mutta niillä ei pystynyt kirjautumaan sisälle. Jos haluaa viilata pilkkua, tässä kohtaa voisi ilmoitusten suhteen ottaa huomioon, mitä kaikkea käyttäjä voi yrittää, mutta sovellus ei kuitenkaan antanut kirjautua tyhjillä tunnuksilla mikä on se pääasia.
Sitten loin ohjeita noudattaen admin-tunnuksen, lisäsin kauppaan onnistuneesti pari tuotetta ja palasin peruskäyttäjän tunnuksella katsomaan niitä. Testasin muutkin sovelluksessa olevat toiminnallisuudet (tuotteiden arvioinnin, arviointien lukemisen ja tuotteen etsimisen haulla, sekä adminilla tuotteiden poistamisen) ja kaikki tuntuvat toimivan asianmukaisesti. Sessiot toimivat (kirjautumisen jälkeen pääsivulla näkyy käyttäjän nimi) ja sovelluksen ulkoasuakin on näköjään jo vähän säädetty. Keksi vielä nettikaupalle nimi ja lisää se vaikka kuvalogona etu- ja pääsivulle.
Kävin myös lukemassa kooditiedostoja ja kaikki näyttää asianmukaiselta. Koodi on jäsennetty selkeästi ja metodeilla on kuvaavat nimet. Jonkin verran siellä oli myös selventäviä kommentteja, hyvä!
tables.sql-tiedostossa oli asianmukaisesti määritellyt tietokantataulut ja sarakkeiden nimet olivat kuvaavat. Tässäkään ei mitään huomautettavaa.Tehtävänannon mukaisesti yritin kovasti keksiä, miten sovellusta voisi parantaa. Toki sieltä puuttuu vielä ominaisuuksia, mutta listasit itsekin toiminnot jotka tulet vielä toteuttamaan. Ja suoraan sanottuna olin helpottunut, ettei kaikilla muillakaan ole sovellus viimeistelyä vaille valmiina, koska omanikin on vielä ihan vaiheessa :). Pop-up -ikkunat ovat ulkoasultaan vähän virheilmomaisia, joten jos voit muokata niiden ulkoasua, se voisi olla yksi säädettävä asia. Mutta tämä on jo ihan pilkun viilaamista ja ei missään nimeessä sovelluksen toiminnallisuuksiin vaikuttava asia.
Sovellus näyttää tässä vaiheessa oikein hyvältä ja toiminnot toimivat niin kuin pitääkin. Kyllä tämä deadlineen mennessä valmistuu. Tsemppiä viimeisille viikoille!