Skip to content

Conversation

@gdostie
Copy link
Collaborator

@gdostie gdostie commented Dec 30, 2025

Proposed Changes

This pull request makes a minor update to the cardsProps definition in Card.figma.tsx, changing the instance name for the children property to match the correct Figma component.

  • Updated the children property in cardsProps to use figma.instance('Swap Content') instead of figma.instance('Content Swap') to reference the correct Figma component.

Potential Breaking Changes

Acceptance Criteria

  • The proposed changes are covered by unit tests
  • The potential breaking changes are clearly identified
  • README.md is adjusted to reflect the proposed changes (if relevant)

@github-actions
Copy link

@github-actions
Copy link

@gdostie gdostie marked this pull request as ready for review December 30, 2025 16:53
@gdostie gdostie requested a review from a team as a code owner December 30, 2025 16:53
@gdostie gdostie requested review from FelixBlaisThon, GermainBergeron and Copilot and removed request for a team December 30, 2025 16:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the Figma Code Connect configuration for the Card component by correcting the instance name reference to match the actual Figma component.

  • Updated the children property in cardsProps from figma.instance('Content Swap') to figma.instance('Swap Content') to align with the correct Figma component name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gdostie gdostie merged commit 947c0b3 into master Jan 6, 2026
15 checks passed
@gdostie gdostie deleted the DS-23-update-card-code-connect branch January 6, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants