Engels

[waaghals]

Engels

[Autsider666]

that's waar, i will letten on it

[Koozza]

Die van mijn are still in half engels en half Dutch ^^' Aanpassen mag :P

[War0n]

Democontroler moet even gecheckt worden. Daarnaast op regel 110 is raar. heb deze niet vertaald omdat er dingen uit gecomment zijn.
Daarnaast Stage.php en Account.php kan ikdie zomaar vertalen? Zelfde geld voor voorbeeldntitieit 1 en 2 .

voor de rest allemaal klaar

[Koozza]

Ja mag je renamen, daarom moest je het zsm doen ;)

Voorbeeld entitie 1/2 en Democontroller hoefde niet echt aangezien dat maar voorbeelden zijn :P

[War0n]

Done :)

[War0n]

Kan jij even mergen?

[Koozza]

Wat moet ik mergen? Geen verstand van, kan Pat denk ik wel

[waaghals]

• Het gemaakte werk moet voldoen aan de acceptatie criteria van de user story N.v.t.
• De broncode van het gemaakte werk heeft unit tests. N.v.t.
• De broncode wordt gereviewd door twee anderen teamleden die het te reviewen werk niet gemaakt hebben.
• Deployment script is getest en werkend met de nieuwe functionaliteit.
• Database wordt ook geupdate
• Documentatie is bijgewerkt.

Zodra een nieuwe functie voldoet aan de DOD mag de nieuwe functionaliteit in de DEV Branch worden gemerged

[War0n]

Is deze gevreviewd?

[Koozza]

19e200b#commitcomment-5547524

Staan overal comments tussendoor en onderaan?

[waaghals]

Ik wilde dit gaan reviewen maar dit is nog niet klaar!
Er zijn nog onvertaalde entities, de relaties kloppen niet en de gegeven feedback is ook nog niet verwerkt.

Doe dat maar eerst voordat ik hier nog meer energie in ga steken.

En geef de commit messages een inhoud. Die hebben we niet voor niets, ik wil kunnen lezen wat je hebt gedaan zonder de diff te bekijken.

[waaghals]

Ding Dong @War0n
Wanneer is dit af?

[War0n]

Vanmiddag:) Sourcetree wil niet op linux, heb hem thuis op men pc goed staan alleen moet hem dus nog pushen enzo.

[War0n]

Dennis Dolman #29 heb de eerste "feedback" verwerkt... Klopt het zo een beejte?

[waaghals]

Review

Gereviewed door: Patrick
Onderwerp: f/translations
Gemaakt door: Dennis

Yes/No Aan alle criteria voldaan

Beoordelings punten

Nummer	Criteria	Voldaan
1	Voldoet de code aan de algemene regeles beschreven in de stijlgids	Yes / No
2	Werkt de code op een correcte manier met andere classes, interfaces en methodes	Yes / No
3	Zijn de methodes logisch en makkelijk aan te passen geschreven	Yes / No
4	De code is consequent geschreven en heeft dezelfde structuur	Yes / No
5	Is het duidelijk wat ieder blok code doet(doormiddel van naamgeving en comentaar)?	Yes / No
6	Voldoet de code aan alle Requirements	Yes / No
8	Is de code secure	Yes / No

Feedback

1. De map view moet dan volgens de nieuwe standaard worden views.
   Tevens is de namespace van de controllers niet hernoemd, hiermee kan de autoloader niet overweg en dat had je zelf ook moeten zien zodra je de voorpagina probeerde te laden. de namespace en de mappen moeten de zelfde namen hebben.
2. Hier heb ik nochYes noch no gegeven. Dit werkt wel maar dat is meer geluk dan wijsheid.
   Even wat toelichting: Er staat in review.php op regel 27. project, omdat het in de zelfde namespace is en er geen andere project klasse is heeft de autoloader er geen probleem mee.
   Maar dit hoeft niet zo te blijven en kan in de toekomst conflicts geven. Tevens gewoon slordig want in de rest wordt gewoon de volle namespace gebruikt.

[Koozza]

Ik sluit me aan bij de review van @waaghals, had de zelfde fouten gevonden en een deel aangegeven in de code zelf.

[War0n]

Feedback is verwerkt*

[waaghals]

Finish maar, wel zorgen dat daarna alles nog werkt svp

[War0n]

werkt