Week 10 (04‐05 tot 08‐05) - nadia2603/stage-maykin GitHub Wiki

04-05

Vandaag ben ik begonnen met het uitwerken van mijn dag van vrijdag. Daarna kreeg ik feedback op mijn pull request van de category list, die heb ik meteen verwerkt. Sven had een vraag over hoe ik iets had gedaan. Ik heb dit uitgelegd en hij gaf aan dat het ook op een andere manier kon, wat beter was omdat de code dan specifieker was. We hebben hierover via Slack gecommuniceerd, omdat Sven thuis werkte. Uiteindelijk is de pull request goedgekeurd en heb ik deze gemerged.

Daarna ben ik begonnen met de breadcrumbs op de category page. Dit duurde langer dan verwacht. Ik had er een hele dag voor gepland, maar dacht eerst dat het in een halve dag zou lukken. Ik moest goed kijken hoe ik de HTML opbouwde, omdat deze verschilt tussen mobile en desktop. Ook heb ik gekeken welke classes ik kon hergebruiken en waar ik nieuwe classes nodig had.

Toen ik klaar was, heb ik een pull request gemaakt en feedback gevraagd. Ik kreeg feedback en heb de code aangepast. Daarna heb ik erbij gezet wat ik heb veranderd en de review opnieuw gestart, zodat er opnieuw naar gekeken kon worden.

In de ochtend was er ook nog een brandoefening. Iedereen moest naar buiten en we moesten vanaf de negende verdieping naar beneden. Toen we buiten waren, mochten we al snel weer naar binnen en konden we verder met het werk.

05-05

Vandaag geen stage. Is bevrijdingsdag.

06-05

Vandaag ben ik begonnen aan de intro van de detail/category page. Ik kon veel code overnemen van de homepagina, omdat de pagina’s erg op elkaar lijken. Ik moest wel kleine dingen aanpassen zodat het beter bij het design past.

Toen ik daarmee klaar was, heb ik een pull request gemaakt en daarna nog feedback verwerkt van een andere pull request. Vervolgens ben ik begonnen aan de searchbar. Die kon ik bijna helemaal overnemen van de homepagina. Ik moest alleen nog de translate toevoegen, zodat de tekst ook vertaald kan worden.

Toen dat af was, heb ik daar ook een pull request van gemaakt.

Daarna heb ik hulp gevraagd bij het verwijderen van de database, omdat de backender veranderingen had gemaakt aan de database. Dit duurde langer dan verwacht, omdat mijn laptop de nieuwe database eerst niet kon vinden. Daarom is er extra hulp gevraagd. Uiteindelijk is het gelukt en kan ik nu verder werken met de nieuwe database.

07-05

Vandaag ben ik begonnen aan de video list. Het opstarten duurde wat langer, omdat de database gisteren is aangepast en ik eerst moest checken of alles goed werkte. Ik had al een branch gemaakt voor de nieuwe database, maar die bleek nog niet de juiste versie te hebben. Daarom heb ik die branch verwijderd en opnieuw een branch aangemaakt vanuit de juiste versie met de nieuwe database.

Daarna ben ik begonnen met een breakdown-schets en de HTML. Die kon ik grotendeels overnemen van de homepagina en hoefde ik alleen kleine aanpassingen te doen. Toen dat af was, merkte ik dat de SCSS met de classes niet goed werkte. Ik kwam erachter dat de branch waarvan ik mijn branch had gemaakt achterliep met de SCSS. Daarom heb ik een rebase gedaan met de main, zodat ik weer de juiste SCSS had in mijn branch.

Toen dat gelukt was, moest ik nog een paar dingen aanpassen zodat alles goed werd weergegeven. Ik zag namelijk dat als er maar één video in stond, deze de volledige breedte innam, ook op desktop. Dat heb ik aangepast zodat het er nu goed uitziet.

Daarna ben ik begonnen aan de pagination. De HTML staat erin, maar het werkt nog niet omdat de backend nog niet af is. Ik had de pagination al eerder gemaakt en zag dat er nog een bug zat op mobile. Daar heb ik samen met Sven naar gekeken. Hij liet mij zien wat de opties waren en wat wel en niet werkte. Uiteindelijk moet ik zelf de oplossingen bedenken en onderbouwen waarom iets wel of niet werkt. Sven heeft me geholpen met de code en heeft de code uitgelegd. Het werkt nu alleen nog niet goed als er maar één ellipsis is. Daar ga ik morgen mee verder.

08-05

Vandaag ben ik verder gegaan met de pagination van de video list. Ik heb gekeken hoe ik de bug kon oplossen en heb meerdere oplossingen geprobeerd en uitgezocht hoe het ook anders kon. Daarnaast heb ik aantekeningen gemaakt over CSS-combinaties en pseudo-elementen.

Ik had uiteindelijk een oplossing gevonden, maar die werkte niet goed wanneer je op pagina 4 zat. Daarom heb ik besloten om voorlopig maar één paginanummer met pijltjes te laten zien.

In de middag kwam een designer langs om te vertellen dat de naamgeving van de kleuren gaat veranderen. Ik had nog geen stylesheet gemaakt, omdat dit op een andere manier gaat dan ik gewend ben. Daarom heb ik besloten om daar later mee verder te gaan. De designer vroeg hoe snel ik het nodig had. Ik gaf aan dat ik bijna klaar ben met de pagina’s en daarna alleen nog kleine dingen hoef te doen. We hebben afgesproken om ergens in de komende twee weken samen te zitten, zodat zij de kleuren kan uitleggen en ik ze daarna goed kan overnemen.

Ik heb ook nog feedback gevraagd over de pagination. Ik moest kijken naar de website van Maykin om te zien hoe het daar was opgelost. Daarbij zag ik dat ik iets fout deed, waardoor de bolletjes met de paginanummers groter waren dan op de website. Ik heb daarna uitgezocht hoe die code in elkaar zat en heb dit toegepast in Samen Leren.

Toen ik klaar was, heb ik aan Sven gevraagd hoe ik de code het beste kon doorgeven aan de backender, omdat de pagination nog werkend gemaakt moet worden. Sven liet zien dat je hiervoor een pull request als draft kan maken. Dat heb ik gedaan en daarbij uitgelegd wat ik heb gemaakt, met foto’s erbij en een overzicht van wat de backender nog moet doen.