{"action":"opened","issue":{"url":"https://api.github.com/repos/samutamm/katonalle/issues/1","repository_url":"https://api.github.com/repos/samutamm/katonalle","labels_url":"https://api.github.com/repos/samutamm/katonalle/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/samutamm/katonalle/issues/1/comments","events_url":"https://api.github.com/repos/samutamm/katonalle/issues/1/events","html_url":"https://github.com/samutamm/katonalle/issues/1","id":132146077,"number":1,"title":"Koodikatselmointi","user":{"login":"Rochet2","id":468816,"avatar_url":"https://avatars.githubusercontent.com/u/468816?v=3","gravatar_id":"","url":"https://api.github.com/users/Rochet2","html_url":"https://github.com/Rochet2","followers_url":"https://api.github.com/users/Rochet2/followers","following_url":"https://api.github.com/users/Rochet2/following{/other_user}","gists_url":"https://api.github.com/users/Rochet2/gists{/gist_id}","starred_url":"https://api.github.com/users/Rochet2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Rochet2/subscriptions","organizations_url":"https://api.github.com/users/Rochet2/orgs","repos_url":"https://api.github.com/users/Rochet2/repos","events_url":"https://api.github.com/users/Rochet2/events{/privacy}","received_events_url":"https://api.github.com/users/Rochet2/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-02-08T13:48:51Z","updated_at":"2016-02-08T13:48:51Z","closed_at":null,"body":"Projekti ladattu 8.2.2016 klo. 15.02\r\n\r\nJoo-o. Jotain omia apeja jossain eri repositorioissa.. jep jep. Ja kaikki javascriptillä tehty. Itse en ole käyttänyt mitään vastaavaa ainakaan samassa skaalassa. Sekavuudesta suureen määrään lisää sekavuutta.\r\nMutta mennään asiaan.\r\n\r\nKyse oli muistaakseni yhdestä PDF tiedostosta ja nyt täällä on monta :(\r\ndokumentaatio.pdf ei ole PDF ollenkaan, vaan joku tekstitiedosto jonka pääte on vaan .pdf\r\nMiten olisi jos muuntaisit nämä yhdeksi ja oikeaan muotoon? Tai ainakin edes oikeaan muotoon. :) Kai sen voi jättää loppunkin .. hmm.\r\n\r\nKaaviot näyttävät suhteellisen selkeiltä ja oikeellisilta.\r\nMitään .sql tiedostoa en löytänyt .. missä testidatat ja muut? :(\r\nJoko testidataa ja tietokannan luovia tiedostoja ei ole tai toinen tai molemmat ovat hyvin lyhyet tai ne jossakin minulle käsittämättömässä muodossa. Ilmeisesti jonkin laisia \"unit testejä\" kuitenkin löysin - jos oikein ymmärsin.\r\nTästä voisi selittää vähän jossain kun kurssi muuten on PHP:llä otetutettu. Muuten joko ne tulisi tehdä tai niitä tarvitsee lisää tai ne on kunnossa - jonka varmaan itse tiedät paremmin.\r\n\r\nTarkoititko apartment vai apart**e**ment? Kaikkialla lukee jälkimmäinen.\r\n\r\nItse sivulla on aika vähän toiminnallisuutta ja sivuja. Jompaa kumpaa toivoisin itse olevan enemmän. En löytänyt missä eri sivut määritellään eikä sivuja ole navigointipalkissa eikä readme:ssa, joten en löytänyt niitä jos niitä edes oli :(\r\nLöysin joitain viitteitä esim. /apartements/new ja vastaavia jotka näyttivät sivujen reitityksiltä, mutta herokussa ne eivät kuitenkaan toimineet. Sivuja ei siis ole tai ne eivät näy herokussa vielä.\r\n\r\nKoodi itsessään näyttää ihan hyvältä mielestäni."}}