Koodikatselmointi

[ollikehy]

Koodi ladattu 23.4.2018, kello 13:30

• Sovelluksen etusivun ulkoasu on heikohko, mutta muilta osin sovellus on tyylikäs
• Taskeille tagien lisääminen on hyvä ominaisuus, helpottaa taskien erottamista toisistaan
• Taskeja ei pysty tällä hetkellä merkitsemään tehdyiksi tai poistamaan listasta eikä tämänhetkistä taskia pysty muuttamaan
• Koodi on laadukasta ja eri toiminnallisuudet on hyvin eroteltu toisistaan eri luokkiin sekä pakkauksiin
• Sovelluksen tietokantakaavio on oikein hyvä, taulujen väliset suhteet selviävät hyvin
• Ohjelmassa olisi hyvä olla ominaisuus joka listaisi kaikki eri taskit yhdelle sivulle (today, tomorrow ja this week)
• Sovelluksessa on myös käytössä javascriptiä joka hoitaa osaa toiminnallisuutta, en tiedä kuinka tarkoituksellista tämä on python-projektissa

[doc97]

Kiitos palautteesta, huomasin että rikoin pari asiaa mutta pushasin nyt ja ongelmat pitäisi olla korjattu. Aion implementoida taskien etsimistä tagien kautta, josta voisi myös katsoa kaikki taskit. Tomisiko tämä vai olitko odottanut jotain erillistä näkymää kaikkien taskien listaukseen?

[ollikehy]

En edelleenkään pystynyt poistamaan/vaihtamaan current taskia Herokussa, lisäksi nyt muiden taskien lisääminen ei onnistunut lainkaan. Tarkoitin kaikkien taskien listauksella esim "all tasks" näkymää jossa näkyisi kaikki taskit samanaikaisesti, suunnittelemasi tagien perusteella etsiminen toimisi varmasti myös tässä tilanteessa jos siihen sisältyisi kaikki kolme erillistä listaa.

[doc97]

Itsellä toimii... täytyypä vähän debugata :)

[doc97]

Taskit ovat tietokannassa, mutta ei viissin näy sinulla selaimessa. Oletko kokeillut force refreshaa (niin ettei selain käytä cached versioo) Ctrl+F5?

[doc97]

Okay löysin bugin. Kiitos paljon :) (complete a task in 'Today' view to see the rest for now)