Skip to content

feat: Code Connect Improvements#639

Open
maximo-macchi-cb wants to merge 11 commits intomasterfrom
max/code-connect-improvements
Open

feat: Code Connect Improvements#639
maximo-macchi-cb wants to merge 11 commits intomasterfrom
max/code-connect-improvements

Conversation

@maximo-macchi-cb
Copy link
Copy Markdown
Contributor

@maximo-macchi-cb maximo-macchi-cb commented Apr 23, 2026

What changed? Why?

This PR adds missing Code Connect mappings and adds a PR check to verify new components have Code Connect mappings. There are also some prop updates made which are a result of writing up the new Code Connect mappings.

Root cause (required for bugfixes)

UI changes

Testing

How has it been tested?

  • Unit tests
  • Interaction tests
  • Pseudo State tests
  • Manual - Web
  • Manual - Android (Emulator / Device)
  • Manual - iOS (Emulator / Device)

Testing instructions

Illustrations/Icons Checklist

Required if this PR changes files under packages/illustrations/** or packages/icons/**

  • verified visreg changes with Terran (include link to visreg run/approval)
  • all illustration/icons names have been reviewed by Dom and/or Terran

Change management

type=routine
risk=low
impact=sev5

automerge=false

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 23, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 1
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
CODEOWNERS 🟡 See below

🟡 CODEOWNERS

Code Owner Status Calculation
ui-systems-eng-team 🟡 0/1
Denominator calculation
Additional CODEOWNERS Requirement
Show calculation
Sum 0
0
From CODEOWNERS 1
Sum 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

fetch-depth: 100 # TODO: This needs to include the merge-base
- uses: ./.github/actions/setup
- name: Check for Code Connect Files
run: yarn tsx ./tools/ci/validators/validateFigmaCodeConnect.ts
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@maximo-macchi-cb you will need to convert to a plain .mjs file - sync with @sverg-cb on your rebase as he just removed TS from this project

@@ -0,0 +1,5 @@
import React from 'react';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: remove?

imports: ["import { RollingNumber } from '@coinbase/cds-mobile/numbers/RollingNumber'"],
example: () => (
<RollingNumber
font="display1"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

there's not figma component properties? 👀

figma.connect(Stepper, FIGMA_URL, {
variant: { type: 'stepper', platform: '📱 mobile' },
imports: ["import { Stepper } from '@coinbase/cds-mobile/stepper'"],
example: () => (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

won't the steps be invisible? I believe in the stepper component in figma thre is a property that defines the number of steps

}),
},
example: ({ ...props }) => (
<SelectChip {...props} onChange={() => {}} options={selectOptions} value="usd" />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same thing - are the options visible in the code preview window?

@@ -0,0 +1,5 @@
import React from 'react';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants