cmda-bt / pt-course-22-23

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

Week 2 - Joppe Koops #22

Open joppekoops opened 1 year ago

joppekoops commented 1 year ago

Progress report

Week

2

Name

Joppe Koops

Class

Tech 2

Repo link

https://github.com/jkhvacmd209/blok-tech

Progress summary

Ik heb deze week beter geleerd hoe ik git kan gebruiken via terminal. Zo heb ik voor de commit messages ingesteld om mijn text editor te gebruiken. Verder heb ik daarbij op mijn wiki gedocumenteerd hoe ik version management ga doen.

Daarnaast heb ik op github een readme toegevoegd en ben ik begonnen met het schrijven van wat content code. Ik moest eerst nog de wireflow van vorige week inhalen, dus qua code heb ik het nog niet heel ver uitgewerkt.

Resources used

https://cbea.ms/git-commit/ https://git-scm.com/book/en/v2/Git-Branching-Branches-in-a-Nutshell Bron voor het stijlen van file inputs: https://css-tricks.com/almanac/selectors/f/file-selector-button/

samanthavz commented 1 year ago

Hi!

Je repo ziet er goed uit, ook je readme. Keep it up :)

Je hebt het er nog niet in staan, maar uiteindelijk wil je ook een .env bestand in je gitignore zetten. (er zijn gitignore templates voor node). Verder zie ik dat je je index.js in je static heb staan, zelf zet ik die altijd in de root folder.

Voor je wiki zou je de template kunnen gebruiker van de wiki van blok tech, hier is een handige sidebar bijv.

Je zou misschien bij je job stories de gekozen job story dikgedrukt kunnen maken voor wat duidelijkheid. Nu lees je pas over je gekozen job story na wat stukken tekst en een hoop afbeeldingen. (je zou dit ook omhoog kunnen halen). Je hebt bij je requirements aangegeven of het een must have, should have, could have of wont have is (neem ik aan), maar hier is geen uitleg bij. Voor iemand die dit niet kent kan dit dus erg verwarrend zijn.

Ik zie af en toe een paar kleine foutjes in spelling, dus let daar op :)

Verder ziet het er erg goed uit!