Oplossen van merge conflicten en mergen van goedgekeurde PR's

[Rozerinay]

Jeff en Bryan hebben samengewerkt aan een design token PR, waarbij componenten uit de Local bibliotheek zijn gecontroleerd op gebroken links. Voor enkele componenten zijn gebroken links geïdentificeerd. Daarnaast zijn er PR’s die al zijn goedgekeurd, maar nog niet zijn gemerged.

Het doel is om:

• De merge conflicten van de betreffende PR’s op te lossen.
• De goedgekeurde PR’s zonder conflicten te mergen.

Door deze actie willen we de bibliotheek consistent houden en gebroken links oplossen.

Acties

• Merge conflicten oplossen:
  • feat: add logo tokens themes#713 vervangen door feat: tokens for logo themes#960
  • feat: add figure tokens themes#744
  • style: changed not-checked style themes#757
  • style: add community tokens for icon themes#812
• Mergen van bovenstaande PR's waarvan conflicten zijn opgelost.
• Mergen van PR's die al geen conflicten hadden:
  • feat: add new case card color tokens themes#724
  • style: restore simplified tokens for Task List component themes#927 vervangen door feat: tokens for task list themes#961
  • feat: add nav bar tokens themes#930 vervangen door feat: tokens for navigation bar themes#962
• Valideren van de gemergede PR’s: Controleren na het mergen of alle componenten correct functioneren en dat gebroken links zijn opgelost.

Acceptatiecriteria

• Alle PR’s met merge conflicten zijn opgelost en succesvol gemerged.
• Goedkeurde PR’s zonder conflicten zijn gemerged.
• Gebroken links in componenten zijn gecontroleerd en hersteld.
• De wijzigingen zijn gevalideerd en geen nieuwe issues zijn geïntroduceerd.

[eslook]

Ik heb de merge conflicts voor deze opgelost, graag mergen voordat er een nieuw merge conflict voorkomt:

• feat: add figure tokens themes#744
• style: changed not-checked style themes#757

Voor deze heb ik een vraag geplaatst over wat de juiste oplossing is bij het mergen:

• style: add community tokens for icon themes#812

Deze laat ik afwachten tot de rest gemerged is, omdat het een zeer grote change is:

• feat: add logo tokens themes#713

[eslook]

Al het mergen is afgerond, nu kan dit gevalideerd worden door design:

• Gebroken links in componenten zijn gecontroleerd en hersteld.
• De wijzigingen zijn gevalideerd en geen nieuwe issues zijn geïntroduceerd.