Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi 11.4.2016 #2

Open
HeikkiHei opened this issue Apr 11, 2016 · 0 comments
Open

Koodikatselmointi 11.4.2016 #2

HeikkiHei opened this issue Apr 11, 2016 · 0 comments

Comments

@HeikkiHei
Copy link

Koodia ja sivustoa tutkittu kello 23.32

Koodi on selkeää, hyvin jaoteltua ja helposti luettavaa. Controllerit selkeitä ja hyvin eriytetty käyttötarkoitusten mukaan. Sivujako selkeä, tosin alikansioitakin voisi käyttää. Muutamia hyviä ratkaisuja tehty, queryt siistejä ja yksinkertaisia.

Sivusto on vielä kovin staattinen, toiminallisuutta ei päässyt testaamaan. Myös testidataa on kovin vähän.

Ainoa nappi joka vei eteenpäin, "Luo Blogi", palautti errorin
SQLSTATE[22P02]: Invalid text representation: 7 ERROR: invalid input syntax for integer: "new". Oletan että sivu on työn alla ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant