7 November 2022

Vandaag ben ik begonnen met het afronden van het two factor authentication feature, eerst ben ik alle nieuwe functies gaan commenten, daarna heb ik alles getest om te kijken wat er nog kapot was. Wat bleek; de verifyKey functie had een kleine logica bug waarin gekeken werdt of de key gelijk was aan de secret OF de secret was verlopen. Ik had een verlopen secret dus kon ik gewoon inloggen, maar dat was niet precies de bedoeling. De OF moest een EN zijn oftewel; || moest && zijn. Hiermee was dit opgelost, hierna had ik alle translations nagekeken of ik alle strings wel getranslated had en toen had ik mijn changes gepushed naar mijn branch en een merge request er aan geslingerd. Later op de dag heeft Stan mijn screenshots uitgeprint en nog wat dingen erbij geschreven die qua de front end nog aangepast moesten worden.

Hierna ben ik verder gegaan met het supporten van de TrackingUrl attribuut in de ParcelPro api connectie, dit was niet te veel werk maar het was meer begrijpwerk (API doorlezen, de bestaande code begrijpen en testen). Hier was ik niet al te lang mee bezig dus dat was wel positief.

Daarna ben ik verder gegaan met een ander issue, wat betrefde het tonen van de producten in de zending op de zending view pagina. Ik had eerst een serverside datatable hiervoor aangemaakt, maar later zei Stan dat dit niet perse serverside hoefde te zijn, omdat er ook geen pagination in zit, en dit is weer omdat er waarschijnlijk toch geen 100+ producten in een bestelling zullen zitten, mischien in de kleinste gevallen wel maar het is in iedergeval niet te verwachten.