15 November 2022

Vandaag heb ik wat kleine dingen rondom mijn merge request van 2fa afgemaakt, dit was alleen maar wat indentatie aanpassen en daarna heb ik deze merge request klaargezet voor de code review.

Toen ik deze had afgerond, was ik door mijn issues heen, dus vroeg ik aan Stan om een nieuw issue. Hij heeft me toen aangeslag gezet met het toevoegen van retouren aan de automatische acties, automatische acties zijn acties die een gebruiker in kan stellen zodat er wanneer er bijvoorbeeld een bestelling wordt aangemaakt er een interne notitie wordt toegevoegd wanneer het land gelijk is aan NL of iets dergelijks.

Dit was er al voor wanneer er een bestelling werdt toegevoegd, maar niet voor wanneer er een retour werdt toegevoegd dus daar ben ik mee aan de slag gegaan. De vorige developer die dit heeft gemaakt had al rekening gehouden met dat er eventueel nog meer automatische acties erbij moesten komen, dus het toevoegen van een nieuwe was niet bepaald rocket science.

Terwijl ik met dit issue bezig was, zei Stan dat mijn query traag was van het verwijderen van een webshop/product link (11s) dus heb ik dit verbeterd. Origineel had ik twee keer wherehas gedaan op de koppeltabel om te kijken of de product id en webshop id gelijk waren aan de webshop/product id die ik had, maar ik moest eerst de uuid van de webshop en van het product omzetten naar een id dus moest ik die twee nog ophalen en daardoor werdt het best traag. Maar op Stan zijn suggestie heb ik dit sneller gemaakt door eerst de product en webshop op te halen vanuit een bedrijf, en vanuit daar een delete te doen op de link. Hierdoor is de query een stuk sneller geworden.

Daarna ben ik de rest van de dag gevuld met de automatische acties.