Koodikatselmointi 2

[sofvanh]

Koodi ladattu 7.5. klo 13:30

Alkuperäinen suunnitelmasi oli ehkä hieman liian työläs, mutta sovelluksen idea on tosi kiinnostava! Vielä on aikaa supistaa käyttötarinoita ja viimeistellä ne ominaisuudet, mitkä olet toteuttanut. En tässä kommentoi kaikkea puuttuvaa, vaan niitä ongelmia, jotka itse korjaisin loppupalautusta varten.

Sovelluksen käyttö

• Tykkäsin siitä, miten rekisteröitymisen jälkeen login-sivu tarjosi omaa käyttäjätunnustani
• Se, että käyttäjä on heti rekisteröidyttyään admin, on heikkous
  • Testausta varten rekisteröityessään käyttäjä voisi valita checkboxilla, onko tavallinen käyttäjä vai admin
• Miten lisään tililleni rahaa? Jos et aijo implementoida tätä, poistaisin koko osion Account details-sivulta.
• Vedon asetuksen jälkeen ei tapahtunut mitään, en saanut mitään palautetta (onnistui / ei onnistunut / error)
• Potti Oy -linkin tulisi viedä etusivulle
• Event approving oli hyvin tehty!
• Kun lisätään custom osallistujia tapahtumaan, formin pitäisi tyhjentyä lisäyksien välissä
• Bugi: Lisäsin muutaman osallistujan, sitten postasin kommentin, jostan syystä osallistujalistaan liittyi kaikki muutkin mahdolliset osallistujat
  • ...mutta kun hyväksyin tapahtuman, listassa olikin enää vain valitsemani kolme osallistujaa
• Bugi: Bettaus tyhjällä laatikolla, siis että määrää ei määritellä, johtaa internal server erroriin
• Formit kannattaa laittaa käyttämään bootstrappia - se tekee sivusta huolitellun näköisen

Dokumentaatio

• README kannattaa päivittää
• Tietokantakaavio ei lataudu GitHubissa (kannattaa ehkä tallentaa tietokantakaavio kuvana ja tehdä .md-tiedosto arkkitehtuurille), ja turha .uxf-tiedosto dokumentaatiokansiossa

Koodi

• auth/views.py: funktio auth_register: Kommentti TODO'hon: itse vaihtaisin muuttujan error nimeksi esim. message, mielestäni olet muuttujan avulla toteuttanut tiedonkulun hyvin, mutta error-nimi on tosiaan vähän hämäävä kun se errorien lisäksi antaa muutakin infoa
• bets/views.py: funktio bet: Koodissa tapahtuu tallennus tietokantaan, mutta ei validointia, että onko käyttäjällä varaa vetoon, eikä käyttäjä saa tietää, onnistuiko veto vai ei, vaikka koodissa veto näyttää aina onnistuvan
• Turhat init-tiedostot bets- ja tasks-kansiossa
• ja Tasks-kansio pitäisi kokonaisuudessaan poistaa, sillä se on nyt tarpeetonta koodia. Muista myös templates/tasks.
• templates/index.html: kaipaa päivitystä, ja tähän pitäisi linkata layout.html'tiedostossa.
  • layout.html: lukee nyt href="#", joka ei ainakaan omalla koneellani toimi: käytä href="{{ url_for('index') }}"

Hyvin menee, tsemppiä loppukurssiin!