2 November 2022

Vandaag is de dag niet gestart met het controleren van merge requests, maar ben ik meteen aan de slag gegaan met een bug op te lossen. Dit was een best wel kleine bug die werdt veroorzaakt door iets wat ik had gemaakt, want ik had in een blade view een variabele gebruikt die ik vanuit een controller meegaf, maar deze view werd ook nog vanuit een andere plaats gebruikt. Hierdoor kwam er een error als je naar die view ging vanuit de andere plaats.

Ook heb ik een nieuw commando gemaakt om het commando lang:js uit te voeren, maar met een andere bestandsnaam. Normaal zou lang:js het bestand 'messages.js' genereren. Maar als dit hetzelfde bleef dan landde dit bestand nog wel eens bij mensen in de cache, waardoor de translations in javascript niet vertaald werden. Nu heb ik een commando gemaakt die dit commando ook aanroept, maar met de parameter met de bestandsnaam. Nu is het bestand 'messages.{time}.js'.

Hierna heb ik ook nog een kleinere bug opgelost, want er is een input formulier die elke keer als deze geupdated wordt, een post request stuurt. Maar als je heel snel iets veranderde, dan ging het nog wel eens mis. Nu heb ik het zo gedaan dat elke nieuwe request de vorige cancelled als deze nog bezig was.

De rest van de dag heb ik besteed aan two factor authentication toevoegen, ik heb als eerste de two fa data van de user model afgehaald en in een aparte tabel gedaan. Hierna heb ik een optie toegevoegd aan two fa om dit via de email te doen.