cmda-bt / pt-course-22-23

🎓 Project Tech · 2022-2023 · Curriculum and Syllabus
MIT License
30 stars 3 forks source link

Week 1 - Luca Oudejans #25

Open lucaoudejans opened 1 year ago

lucaoudejans commented 1 year ago

Progress report

Week and package

Week 1

Name

Luca Oudejans

Class

TEC 2

Repo link

repo link

Progress summary

Tot nu toe de opdrachten in Notion geplaatst

Resources used

Internet

Any thoughts?

samanthavz commented 1 year ago

Hoi!

Wat feedback:

Wat betreft je code, let op je mappenstructuur. Zo is een backend map niet per se nodig en kan je alle content zoals non server side scripts, css, html, images en files in een static map zetten. Verder zie ik dat je meerdere DS_store's in je repo hebt, deze mogen er niet in dus verwijder die. Je hebt hem wel al in je gitignore staan zie ik. Ook zie ik een mapje van project tech wiki, dit is niet nodig.

Wat betreft je readme, hoewel ik erg fan ben van kaas is dit helaas niet wat de docenten willen zien. Denk hierbij aan een stukje tekst over je project, concept, how to install, contributers etc.

Je wiki is helemaal leeg dus hier kan ik geen feedback op geven, zorg ervoor dat je dit zo snel mogelijk gaat aanvullen!

Succes :)