20 December 2022

Vandaag heb ik een eindsprint getrokken het verplaatslijst issue, ik heb uitgezocht waar er allemaal voorraden van locatie gewijzigd kunnen worden en op die plekken checks geplaatst die controleren of de locatie waar de voorraad heen gaat geen limiet van 1 heeft of nog geen product heeft. Volgensmij heb ik dit op alle plekken gedaan, hierna heb ik. mijn changes gecommit, een merge request gemaakt en nog even door deze merge request heen gekeken of ik geen gekke dingen zag. Zover ik kon zien was alles in orde en heb ik het 'code review' label aan de request gegeven om aan te geven dat het nagekeken kan worden.

Hierna ben ik verder gegaan met een nieuw issue; namelijk het toevoegen van returns aan de fulfilment API. Deze zitten er op de normale API al in maar nog niet op de API voor fulfilment klanten. Terwijl ik wat dingen aan het testen was kwam ik een bug tegen waardoor het bijna niet mogelijk was om via de API een return aan te maken. Na een paar seconden zoeken had ik het gevonden.

Het probleem was dat er een functie werdt aangeroepen die een boolean returned, deze functie heeft ook twee required parameters maar die werden niet meegegeven. Ook werdt er niks gedaan met de return waarde. Nadat ik dit weghaalde was het probleem opgelost, dus heb ik een aparte branch + merge request gemaakt met de fix voor deze bug.