{"action":"created","issue":{"url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/1","labels_url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/1/comments","events_url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/1/events","html_url":"https://github.com/googol/drinkkiarkisto/issues/1","id":117002217,"number":1,"title":"Koodikatselmointi","user":{"login":"heileino","id":14069307,"avatar_url":"https://avatars.githubusercontent.com/u/14069307?v=3","gravatar_id":"","url":"https://api.github.com/users/heileino","html_url":"https://github.com/heileino","followers_url":"https://api.github.com/users/heileino/followers","following_url":"https://api.github.com/users/heileino/following{/other_user}","gists_url":"https://api.github.com/users/heileino/gists{/gist_id}","starred_url":"https://api.github.com/users/heileino/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/heileino/subscriptions","organizations_url":"https://api.github.com/users/heileino/orgs","repos_url":"https://api.github.com/users/heileino/repos","events_url":"https://api.github.com/users/heileino/events{/privacy}","received_events_url":"https://api.github.com/users/heileino/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-15T16:27:52Z","updated_at":"2015-11-15T19:37:47Z","closed_at":null,"body":"Repo/nettisivut ladattu su 15.11.2015 klo 16.10\r\n\r\n- Aloitussivu on saatu tässä kehitysvaiheessa yksinkertaisen selkeäksi ja informatiiviseksi. Kyseistä tapaa voisi käyttää myös lopullisessa versiossa, jolloin etusivulla voisi olla muutama jollain kriteereillä valittuja esimerkkidrinkkejä tietoineen. Kriteereitä voisivat olla esimerkiksi haetuimmat drinkit, sesonkiin sopivat drinkit, ylläpidon suositukset jne.\r\n\r\n- Olemassa olevat sivunäkymät vaikuttavat toistaiseksi staattisilta, eli niiden tiedoilla ei ole vielä yhteyttä tietokantaan. Uuden drinkin luomissivun näkymässä tarjotaan käyttäjäystävällinen tapa käyttää nuolia oikean annosmäärän lisäämiseksi. Näin käyttäjä saa kätevästi pelkän hiiren avulla syötettyä runsaasti lukuja. Samalla kirjoitusvirheiden mahdollisuutta pienennetään. Drinkkien muokkaussivut toimivat samalla periaatteella ja tarjoavat oikeaoppisesti jo olemassa olevat tiedot muokkauksen tueksi. Näkymät tarjoavat hyvän pohjan varsinaisen sivuston luomiseen.\r\n\r\n- Koodin osalta sovelluksen kontrolleri ja näkymät on toteutettu itselleni vieraalla kielellä, JavaScriptillä, joten syvällistä analyysia koodin tasosta on mahdoton antaa.  Päällisin puolin kontrolleri näyttää toteuttavan tehtävänsä drinkkeihin liittyvissä pyynnöissä ohjeistuksessa vaaditulla tavalla. Näkymissä oleva tiedosto server.js näyttää vastaavan routes.php-tiedostoa. Reititykset vaikuttavat valideilta. Muilta osin ymmärrykseni ei riitä näkymien analysointiin, sillä niistä ei löydy tuttuja html-tiedostoja, vaan näkymät on nähtävästi toteutettu jollakin tapaa JavaScript-tiedostojen kautta.\r\n\r\n- Tietokantojen tietosisältö on toteutettu mielekkäästi, relaatiotietokantakaaviota mukaillen. Tietokohteiden attribuutteja on riittävästi tarjoamaan haluttuja tietoja tietokohteistaan. Create-table, drop-Table ja add-test-data tiedostot toteutettu toimivalla ja selkeällä tavalla.\r\n\r\n- Yhteenvetona voi sanoa, että työ on siinä vaiheessa, että paljon on vielä tehtävää ja työn onnistumisen arviointi ei projektin tässä vaiheessa ole mielekästä. README-tiedosto sisältää hienon selvityksen repositoryn kansiorakenteesta ja sen sisällöstä, mikä auttaa myös asiasta ymmärtämätöntä hieman hahmoittamaan työn rakenteen ideaa. Edellä mainitusta tiedostosta puuttuu kuitenkin ohjeistuksessa vaaditut suorat linkit erilaisiin olemassa oleviin sivunäkymiin. Aihekuvauksessa on monenlaisia vaatimuksia, joita ei ole vielä hahmoteltu ainakaan sellaisissa sivunäkymissä, joihin sivuston etusivulta on pääsy. Olemassa oleva pohja vaikuttaa hyvin ammattimaisesti rakennetulta ja tekijä selvästi tietää katselmoijaa paremmin, mitä on tekemässä ja miten hyviä tietokantasovelluksia luodaan."},"comment":{"url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/comments/156845935","html_url":"https://github.com/googol/drinkkiarkisto/issues/1#issuecomment-156845935","issue_url":"https://api.github.com/repos/googol/drinkkiarkisto/issues/1","id":156845935,"user":{"login":"heileino","id":14069307,"avatar_url":"https://avatars.githubusercontent.com/u/14069307?v=3","gravatar_id":"","url":"https://api.github.com/users/heileino","html_url":"https://github.com/heileino","followers_url":"https://api.github.com/users/heileino/followers","following_url":"https://api.github.com/users/heileino/following{/other_user}","gists_url":"https://api.github.com/users/heileino/gists{/gist_id}","starred_url":"https://api.github.com/users/heileino/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/heileino/subscriptions","organizations_url":"https://api.github.com/users/heileino/orgs","repos_url":"https://api.github.com/users/heileino/repos","events_url":"https://api.github.com/users/heileino/events{/privacy}","received_events_url":"https://api.github.com/users/heileino/received_events","type":"User","site_admin":false},"created_at":"2015-11-15T19:37:47Z","updated_at":"2015-11-15T19:37:47Z","body":"http://tsoha.github.io/#/johdanto#top. Tuolla PHP-materiaalisivuilla lukee ainakin viikon 2 palautusvaatimuksien kohdalla näin:\r\n\"Suunnittele käyttöliittymäsi ja toteuta niistä staattiset HTML-sivut. Muista lisätä linkit toteuttamiisi sivuihin reposi README.md-tiedostoon. (1,5p)\r\n- Suunnittele etusivu\r\n- Suunnittele listaussivut\r\n- Suunnittele muokkaus- ja esittelysivut\"\r\n\r\nTärkeää arvostelun kannalta varmasti erityisesti silloin, jos kaikki sivunäkymähahmotelmat eivät ole vielä navigoitavissa etusivun kautta."}}