Koodi otettu talteen 24.4. klo 17

Sovelluksen käyttö

Koko sovellus:

• Sivut voisi olla jäsennelty paremmin (otsikoitu)
• SetlistApp ja List songs-linkit vie samalle etusivulle

Etusivu:

• Change status-nappi on näköjään kaikkien kirjautuneiden käytössä, ei vain ylläpitäjän, eikä statuksen muutos vielä tee mitään

Settilistat:

• Olisi hyvä pystyä lisäämään lauluja settilistaan myös täältä, settilistan omalta sivulta
• Hienot layoutit lista- ja details-sivuilla
• Settilistasta pitäisi voida jotenkin tietää, onko se oma

Autorisointi:

• Myös login- ja sign up-formit voisi käyttää bootstrappia

Koodin laatu

• songs-kansiossa ilmeisesti turha, tyhjä init-tiedosto
• songs/forms.py: choices-lista voisi ehkä olla määritelty vain kerran staattisena listana, ja metodit voisi viitata siihen luettavuuden parantamiseksi

Hyvin menee, tsemppiä loppukurssiin!

7.5.2018 klo 19:00

Hei,

• Ihan toimiva sovellus yleisesti. Koodi on selkeästi kirjoitettu ja parametrit ja muuttujat tarpeeksi hyvin laadittu. Koodia helppo lukea. Koodissa voisi lisätä ns. "main" views.py mistä koodi lähtee rakentumaan. Nyt piti löytää ja etsiä oikea views.py ja katsoa mistä ns. alkaa.

• Koodiin voisi kirjoittaa paljon enemmän kommentteja, vaikkapa metodejen alkuun, että mitä kyseinen metodi tekee, niin siitä on helppo seurata mitä siinä tapahtuu, kun pystyy jo ennakoimaan hiukan mitä tulee tapahtumaan.

• Koodin, luokkien ja tiedostojen jakaminen omiin kansioihin ja omiin osiin on totoetettu hyvin.

• Kappaleen pituuden laittamisessa voi olla ongelmia jos ei tiedä mitä siinä haetaan. Esim. onko pituus sekuntteina, vai annetaako minuutit ja sekunnit erikseen yms. Kappale listauksessa tämän kyllä huomaa, mutta ei uutta kappaletta luodessa.

• Kappaleet voisi olla ehkä listattuna selkämmin, että mikä on oma kappale ja mikä jokun toisen. Tykkäisin ehkä itse jos käyttäjällä olisi jonkinlainen omasivu, mistä näkisi selkeästi omat listat yms. Muuten listaus näkymä on hyvä.

• sivuston etusivulla voisi olla jonkinlainen intro teksti mikä sivusto tämä on, esim jos tuntematon käyttäjä eksyy sivulle ja hänelle annetaan heti vain että login. Hänen täytyy tietää mikä tämä sivusto on.

• Sivulla on hyvät validointi tekstit. Sivustola voisi olla myös jonkin lainen admin/superuser, joka voisi pitää sivustoa silmällä "perseilyltä".

• Soittolistojen katselmointi selkeä, omaa soittolistaa luodessa esimerkki minkä olet öaittanut näkyviin on mielestäni hyvä/kiva. Omia soittolistoja katsellessa voisi ehkä olla myös samallainen katselutila, kuin muiden soittolistojen katselussa ja sitten nappulan josta päästään editointiin.

Älä ota liian vakavasti palautetta. Tämä oli vain mielipiteeni sivustolla ensikertaa vieraillessani, mitä ehkä käyttäjänä haluaisin, mutta sovellus tekee asiat mitä sovelluksen määrittelyssä sanotaa..

Viikon 3 etapeista kaikki, paitsi täysi CRUD tehty.
Kirjautuminen ja tunnuksen luominen toimii.
Kappaleiden lisääminen ja niiden julkiseksi muuttaminen toimii.