Ohjelma ladattu 30.3 klo 22.10.

-README:n linkit keskustelunaiheisiin ja keskusteluun eivät toimineet. Muut linkit toimivat, jote toimimattomien linkkien sivuille pääsi niiden kautta.
-dokumentaatio on hyvin huolellella tehty ja selkeää luettavaa.
-käyttäjät eivät voi itse luoda itselleen tunnuksia käyttötapauskaavion mukaan, onko tämä hyvä keskustelufoorumille? saatat haluta että adminin ei tarvitse luoda kaikille uusille käyttäjille tunnuksia.
-keskusteluiden luomisella tulee varmasti vielä painike sivulle.
-kannattaa kiinnittää huomiota foorumin valmistamisessa sivutukseen jos haluat toteuttaa foorumit joissa esim. sivulle mahtuu kerralla kymmenen aihetta tai viestiä. Tämän toteuttaminen oli meille Tietokannat-kurssilla vaikeaa joten päädyimme itse käyttämään ratkaisua, jossa meillä selain-linkki näytti tältä: /keskustelu/1/sivu/2. Tämä näytti hieman rumalta, mutta pystyimme jakamaan foorumeita useammille sivuille kun se oli mielekästä. Esimerkissä siis keskustelun 1 sivun 2 tarkastelu, jossa ensimmäisellä sivulla vaikka 10 viestiä ja toisella sivulla seuraavat viisi. Nykyisessä muodossaan tietyn keskustelun kaikki viestit näkyisivät samalla sivulla kaiketi?
-Siirtääkö nykyisessä koodissa oleva viestin luonti käyttäjän keskustelut-osioon? Jos näin, olisiko hyvä päätyä takaisin samaan keskusteluun jonne viesti kirjoitettiin.

public static function addViesti($id){
	$params = $_POST;
	$viesti = new Viesti(array(
	  'keskustelu_id' => $id,
	  'kayttaja_id' => 1,
	  'sisalto' => $params['content']
	));
	
	$viesti->save();
	Redirect::to('/keskustelut');   // **tämä tässä**
}

-On hyvä että keskustelunalueella lisättäessä uusi keskustelu täytyy keskustelunaloittajan lisätä myös viesti. Pystyykö admin poistamaan keskustelusta ainoan sen sisältämän viestin jättämällä tyhjän keskustelun?
-Relaatiotietokantakaavio, käsitekaavio ja käyttötapausdiagrammi hyvän näköisiä :)
-Kirjoitusvirheitä dokumentaatiossa: Tietokohde Viestissä: sisältä ja vastaa? "lukijalla näytetään automaattosesti"-johdannossa.
-Hieno homma, mukavaa jatkoa!

Repo ladattu klo 22.47 22.04.2017

Aihevalintana foorumi on ehkä hieman tylsä ottaen huomioon, että tietokantojen perusteissa on jo toteutettu foorumi(?). Toki tämä foorumi on toteutettu php:llä, mikä ei ole aivan yksi yhteen perusteiden kanssa.
Perustoiminnallisuus foorumissa vaikuttaisi olevan hyvällä mallilla. Viestit tallentuvat tietokantaan oikein ja näkyvät myöskin oikein. Valitettavasti viikon 5 palautusta ei ole tehty, joten esimerkiksi sisäänkirjautumisen kanssa on vielä tekemistä. Tällä hetkellä en pysty kirjautumaan sovelluksesta ulos, vaikka käyttäjän vaihtaminen onkin mahdollista. Myös dokumentaation ja muiden viikon 5 vaatimusten osalta projetki vaatii luonnollisesti vielä tekemistä.
Näkymät on tällä erää enimmäkseen "suunnitelmat"-kansiossa. Tässä vaiheessa näkymät olisi syytä olla jaettuna omiin kuvaavasti nimettyhin kansioihinsa.
forums_controller.php tiedostosta löytyy kolme destroy-metodia, destroy, destroya, destroyk. Nämä voisi olla kuvaavammin nimettyjä.
Yhteenvetona projekti täyttää tekniset vaatimukset viikkoon 4 asti koodin ja dokumentoinnin osalta. Rehellisesti sanoen minulle jää siitä huolimatta sellainen kuva, että tässä pyritään menemään yli aidan matalimmasta kohdasta. Vielä ei kuitenkaan ole liian myöhäistä sankarikoodata 5/5 projekti, joten tsemppiä loppupalautukseen!