31 Oktober 2022

Vandaag ben ik weer begonnen met mijn merge request, de merge request van de product webshop links verwijderen had nog commens maar dit bevatte meer kleinere dingen zoals double quotes naar single quotes veranderen of een spatie na de if.  Hierna had ik nog een kleine bugfix gedaan, er was namelijk een datatabel op een pagina waar serverside; false was, maar door een aanpassing van mij heeft elke datatabel default ajax code. Maar op een lokale datatabel veroorzaakt dat een error, Glenn had hier al een oplossing voor gevonden maar wat bleek is dat je gewoon ajax: null kon doen om deze default code ongedaan te maken.

Hierna heb ik een ander merge request afgerond, waar nog een comment in stond. Dit was de merge request van tonen hoeveel producten er in een picklijst zullen zitten bij het aanmaken, Stan wilde dat er terwijl dat er werdt opgehaald hoeveel er in de picklist zouden zitten, er een laad icoontje tevoorschijn kwam.

Toen ik alle merge requests had afgerond, ben ik verder gegaan met 2fa toevoegen. Ik heb dit aan Stan laten zien, en hier kwamen wel een aantal punten uit; Hierna ben ik dus begonnen aan het toevoegen van een tweede optie voor 2fa waar je ook via de email een code kan ontvangen.