rowinruizendaal / progressive-web-apps-2021

The course Progressive Web Apps is about learning to build server side rendered applications. Progressive Web Apps is part of the half year minor programme about Web Design and Development in Amsterdam. Bachelor Communication and Multimedia Design, Amsterdam University of Applied Science.

Hi there 👋

Tech and Tools

Current favourites

Other tools I use

"The best error message is the one that never shows up."

progressive-web-apps-2021's Issues

Code review week 1

Yo Rowin!

Goed dat je hebt gedacht aan deployen, alleen gooit ie een error. Even naar kijken misschien ;)

Wat je readme betreft, ik denk dat je heel veel kan gebruiken uit je WAFS readme, maar niet alles. Kijk wat relevant is. Dingen die je kan toevoegen zijn: Een stukje over je packages. Welke packages gebruik je en waarom? Beschrijf dit en een linkje naar de npm docs zou niet misstaan. Hetzelfde geldt voor je scripts. Momenteel heb je er twee die redelijk voor zichzelf spreken, maar als je er meer aan toevoegt, is het handig om even te omschrijven wat voor keuzes je hebt gemaakt en hoe je script chain in elkaar steekt.

Goed dat je je routes hebt gescheiden, alleen zou ik ze even in een variable zetten, zo hou je je requires gescheiden van de rest van je code en het wordt wat duidelijker.

Je templates zijn netjes gescheiden, goed bezig!

Kijk nog even naar wat meer tooling. Als je geen idee hebt, kijk even bij je teamgenoten en zoek op wat ze gebruiken.

Succes!

Converting WAFS to server-side rendering

Feedback on converting to server-side rendering

General feedback regarding assignment

• ✔️ Website is deployed server side on Heroku, and is functioning
• ✔️ Used Express router to split routes, you understand how Express works
• ⚠️ Could look into some more tooling

Some things I think can be easily resolved

• I would recommend sticking to English variable names if the rest is all English:
  

  progressive-web-apps-2021/routes/src/index.js

  Lines 93 to 94 in 19266bf

  	if (json && !json.error) {
  	const leeg = [];

• If you want to write more ES6 code, it might be worth looking into something like Babel to be able to convert this to 'import' and 'export':
  

  progressive-web-apps-2021/routes/src/index.js

  Lines 1 to 3 in 19266bf

  	const express = require('express');
  	const router = express.Router();
  	const axios = require('axios');

• You're mixing async await & promise.then here, you only need to use one of them. Since it's already an async function, you can just use the await:
  

  progressive-web-apps-2021/routes/src/index.js

  Lines 33 to 38 in 19266bf

  	json = await axios.request(options).then(function(response) {
  	data.push(response.data);
  	return response.data;
  	}).catch(function(error) {
  	console.error(error);
  	});

I've seen this multiple times in your code, you're doing double syntax which is unnecessary.

• I think it would be more neat to do the require/import at the top here:
  app.use(require('./routes/router'));