Skip to content

Conversation

e412
Copy link
Collaborator

@e412 e412 commented Dec 3, 2019

No description provided.

@e412 e412 requested review from NoraAnlima and roschaefer December 3, 2019 22:27
@e412 e412 assigned SyJuly and e412 Dec 3, 2019
Copy link
Collaborator

@NoraAnlima NoraAnlima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Voll cool ihr habt ja schon angefangen eine Loginpage zu bauen. Habt ihr da schon ein bisschen rumgespielt mit der Integration und schon ein paar gute Tipps? 😃

Noch eine andere Anmerkung: Wir haben die ganze Aufgabe 3 bei uns in ein eigenes "Projekt" ausgelagert um die Vermischung der Abhängigkeiten zu verhindern und das richtige starten von Backend und Frontend zu ermöglichen, da die beiden in ihrer Ausführung ja doch sehr unterschiedlich sind (der Frontend Code wird ja per HTTP an den Client geschickt und der Backend Code bleibt ja auf dem ausführenden Server und ist nur eine API also eine Art Service). @roschaefer kann uns hier aber bestimmt einen Tipp hinsichtlich einer guten Projektstruktur für diesen Fall geben :).

Und zum Schluss noch das obligatorische Wieder-eine-Aufgabe-geschafft-gif:

nXoliun

@e412
Copy link
Collaborator Author

e412 commented Dec 13, 2019

@roschaefer wir würden uns über dein Feedback freuen, damit mir dem PR jetzt am WE in den Master mergen können:)

@SyJuly SyJuly merged commit 14db86b into master Dec 14, 2019
SyJuly pushed a commit that referenced this pull request Dec 14, 2019
@roschaefer
Copy link
Collaborator

@e412 sehr schön! Ich habe euren PR in #5 ge-reviewed und das review von @NoraAnlima eingetragen.

@NoraAnlima Ich persönlich habe mal gehört, dass das directory layout eines repositories kommunizieren sollte, welche Anwendung es enthält.

Deswegen haben wir es bei uns so gemacht:

webapp/
backend/
android/ - fehlt noch
ios/ - fehlt noch
deployment/
cypress/ - full stack tests

Wenn ich selber die Hausaufgaben lósen würde, dann würden bei mir im repo die Aufgabenbeschreibungen auf der selben Ebene zu finden sein wie z.b. backend/ und webapp/.

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

Successfully merging this pull request may close these issues.

4 participants