21 December 2022

Vandaag ben ik begonnen met het nakijken van mijn merge requests, op de merge request van de verplaatslijst die ik gisteren had opengezet stonden nog een paar comments open. Deze betrefde meer het formaat van de code, maar een ding wat daar buiten viel was dat ik op een array [0][0] deed om eerst het eerste item van de array te krijgen, en daarna [0] voor de id, maar ik heb deze array in de javascript omgezet naar een javascript object met duidelijke keys voor elke waarde zodat het wat duidelijker werdt.

Daarna heb ik nog een ander merge request nagekeken van het automatisch uitloggen na x aantal seconden, maar omdat two factor authentication gemerged was naar develop moest ik het even testen of het nog goed zou werken in combinatie met two factor authentication. Dit werkte allemaal wel prima op een puntje na, namelijk de redirect werkte niet als je two factor authentication had ingesteld. Dit heb ik opgelost door ook op die view de redirect url mee te geven in een hidden URL.

Daarna heb ik een start gemaakt aan de bestellingen import doormiddel van een excel bestand.