Full Stack projektin katselmointi

[Jakousa]

Full Stack projektin katselmointi

Tässä lyhyt katselmointi Full Stack kurssin projektistasi. Katselmoinnissa esitetyt kommentit ovat parannusehdotuksia, joita voit halutessasi ottaa huomioon tässä tai tulevissa projekteissasi. Mitään muutoksia tähän projektiin ei siis ole pakko tehdä suoritusmerkintää varten.

Käytettävyys

Mitä tein?

Rekisteröidyin

Kirjauduin sisään

Kommentoin

Kokemus

Favicon ja title voisi vaihtaa, ja voisit kikkailla footerin sivun alareunaan vaikka sivulla ei olisi sisältöä.

Koodi

Hyvän näköistä ja jaettu järkeviin kokonaisuuksiin.

Else returnin jälkeen turha.

fullstack-spaceapp/controllers/users.js

Lines 9 to 12 in 197d70b

	if (body.username === null) {
	return response.status(400).json({ error: 'Choose a username.' })
	}
	else if (body.password === null) {

Commitoitu vahingossa mukaan jotain?

fullstack-spaceapp/controllers/comments.js

Lines 60 to 77 in 197d70b

	/* // below is copy-paste from elsewhere, not usable as such!!
	notesRouter.put('/:id', (request, response, next) => {
	const body = request.body
	const note = {
	content: body.content,
	important: body.important,
	}
	Note.findByIdAndUpdate(request.params.id, note, { new: true })
	.then(updatedNote => {
	response.json(updatedNote.toJSON())
	})
	.catch(error => next(error))
	})
	*/

En löytänyt frontin koodia..?

Kannattaa tutustua kontteihin esimerkiksi https://devopswithdocker.com kurssilla, siellä opitaan miten paketoidaan websovellukset järkeviin paketteihin julkaisua varten. Säästyt tuolta minifioidun koodin commitoimiselta.

Kokonaisuus

Hieno projekti ja hyvää työtä sen kanssa!