xneme / tsoha-vedonlyonti Goto Github PK

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!

Latasin tiedostot ti 24.4. klo 11:42.

Keskeneräisyydestään huolimatta mielenkiintoinen projekti! Osa kommenteistani saattaa kuulua luokkaan "on noin, koska keskeneräinen, duh!", mutta listailin silti kaiken, mitä silmiini osui. Kävin siis herokussa testaamassa event-listausta ja uusien eventtien ehdottamista, poistamista (joka ei tuntunut toimivan) ja kommentointia. Kommentointi lisäsi kommentin sivulle, mutta jätti kirjoitetun kommentin sivun päivityksen yhteydessä formiin. Pitäisiköhän formi samalla tyhjentää?

Sivustosta yleisemmin: nimeämisessä hiukan sekoittaa se, että bets ja events viitannevat samaan asiaan. Sivustollakin "List events" osoittaa polkuun https://tsoha-vedonlyonti.herokuapp.com/bets/
Samoin polut /bets/<event_id>/ jne. Kannattaisikohan muuttaa betsit eventeiksi, vai ymmärsinkö niiden funktion väärin?

Miten Events muuten suhteutuu tietokantakaavioon? Kaaviossa oleva Contest vaikuttaisi olevan sama asia kuin Event. Siis onko Bets == Events == Contests? Antaako aihetta nimistön päivittelyyn?

Tällä hetkellä eventin poistaminen edellyttää eventin avaamista. Ehkä olisi parempi, jos admin voisi poistaa eventin jo eventtilistauksesta? Sama juttu saattaisi toimia eventtien hyväksymisen suhteen.

Mitä jos eventtilistauksessa näytettäisiin myös kiinnostuneiden ja/tai kommenttien lukumäärä, niin kiinnostavat/suositut eventit nousisivat esille jo listauksessa ja niiden mukaan voisi kenties myös sortata?

Käyttäjätapauksissa ja aihemäärittelyssä ei ainakaan erikseen mainita, että käyttäjä voisi tarkastella omia tietojaan. Kun kuitenkin Accountissa on "balance", niin kannattaisikohan projektissa olla käyttäjälle oma sivu, josta näkee, paljonko rahaa on jäljellä?

Yksityiskohtana ulkoasusta: tervehdys + käyttäjän nimi on lähes-mustaa mustalla pohjalla, joten se näkyy aika heikosti.

Koodista osui ainoana silmään, että validoinnit huomioivat tällä hetkellä syötetyn merkkimäärän alarajan, mutta eivät ylärajaa.

User storyista sen verran, että ensimmäinen niistä voisi olla jaettavissa kahteen omaan storyynsä: Asiakas voi luoda tilin. Asiakas voi tallettaa rahaa tilisiirtona. Lisäksi kaikkia aihemäärittelyssä olevia toimintoja (kuten uuden salasanalistan pyyntö) ei löytynyt user storyjen joukosta.

Tsemppiä projektin jatkoon!