14 November 2022

Vandaag ben ik weer begonnen met de merge requests, er stonden er nog een paar open waar ik de puntjes op de I heb gezet namelijk; het issue van producten op de zending pagina tonen. Deze comments van Stan betrefde meer kleinere dingen dan grote, en de comments van Glenn waren optimalisatie doormiddel van een with(). Hierna heb ik het issue van de purchase orders api endpoint voor fulfilment afgerond, hier waren best veel comments op. Een groot deel van deze comments betrefde een error code 404 veranderen naar 400, daarnaast was het wat naamgeving veranderen, wat indentatie aanpassen en nog een klein logica dingetje omtrent het aanpassen van de status van een purchase order naar cancelled als er al dingen zij besteld.

Hierna ben ik verder gegaan met het issue van meer demo data toevoegen, hier heb ik nu een extra webshop aan toegevoegd, zendingen en meer bestellingen. Toen ik daar mee klaar was heb ik nog even wat punten opgenomen in het issue van de two factor authentication, een punt van Stan was dat als je het wilde verwijderen, je tweestapsverificatie token niet nodig zou hebben. En toen ik dit had afgerond en gepushed was het al weer 5 uur.