4. Integreren 🌍 - Daniquedejong/visual-thinking GitHub Wiki

Live zetten Danique

Badr heeft natuurlijk al zijn eigen live link, maar ik niet. Dus dit probeer ik via vercel te doen, maar ik krijg wat errors.

Ik kwam er achter dat ik de hygraph url's nog niet goed had doorgegeven.

Gelukt!

Dit is nu mijn link:

• https://visual-thinking-k1e8.vercel.app/

Pull-request errors

Wat ging er fout?

We hebben helaas wat problemen met pull-requests ervaren. Ik (Danique) had een PR gemaakt voor twee taken, wat al niet helemaal de bedoeling is, en hierna heb ik nog veel commits in de PR gedaan. Ik had niet helemaal door dat deze zich bij de PR aan zouden sluiten, omdat ik dacht dat ik weer in de release-candidate werkte. Dit was dus niet het geval, dus de PR werd als maar groter met nieuwe commits.

Joost probeerde uiteindelijk deze PR te mergen naar de release-candidate van de agency, maar hij kreeg allerlei errors. Het kwam erop neer dat mijn PR zo ver achter liep op de destijds huidige release-candidate van de agency, dat ik de hele code om zou moeten gooien om die PR te laten mergen.

Hoe zijn we tot een oplossing gekomen?

Samen met Joost heb ik gekeken naar alle errors. We hebben op dat moment besloten dat het beter was om deze PR te closen, dus niet mergen, en hem op te delen in allemaal nieuwe kleine PR's.

Uitwerking

Ik heb de grote PR uit elkaar gehaald en allemaal verschillende kleine branches aangemaakt. Hier heb ik een voor een aan gewerkt om er vervolgens losse PR's van te maken. Wanneer Joost er een had gemerged naar de agency, heb ik alle data weer gefetched en mijn fork gesyncd. Op deze manier bleef mijn eigen release-candidate steeds up-to-date met die van de agency (en dus ook Badr's PR's) en kon er niks meer fout gaan.

Schetsen

Dit was een aardig probleem, wat erg gecompliceerd in elkaar zat. Ik ben even gaan zitten om dit zo duidelijk mogelijk te schetsen. Aller eerst heb ik een losse schets op papier gemaakt. Hierna heb ik deze uitgewerkt in twee figma schetsen.

Fout

In deze 'foute' schets zie je wat er eerst gebeurde en waar het fout ging. Je kunt zien dat Badr steeds zijn PR's kon mergen via Joost en dat hij alles gelijk weer syncte en had gefetched. Hierdoor bleef hij up-to-date. Bij mijn branches daarentegen ging het dus goed fout. De commits stapelde zich op en ik liep erg achter op de agency code.

Goed

In de 'goede' schets kun je zien hoe het nu is opgelost en dat ik het beter begrijp. Ik heb de branches veel beter opgesplitst en probeer het bij 1 commit te houden per branch. Ook blijft mijn release-candidate nu steeds up-to-date met de agency code.

Conclusie

Nu heb ik een veel beter zicht op hoe de branches in elkaar zitten. Ik dacht dat ik het al begreep, maar blijkbaar moest ik het toch even meemaken om er echt verstand van te krijgen.

Ik heb samen met Badr gezeten om uit te leggen wat er allemaal aan de hand was met mijn pull-request. Door deze schetsen te laten zien begreep hij goed wat er gebeurd was.