Skip to content

Fix duplicate keys in DesignableBanner#15134

Merged
jamesmockett merged 1 commit intomainfrom
jm/designable-banner-dup-key
Jan 15, 2026
Merged

Fix duplicate keys in DesignableBanner#15134
jamesmockett merged 1 commit intomainfrom
jm/designable-banner-dup-key

Conversation

@jamesmockett
Copy link
Contributor

What does this change?

Switches to using index rather than image URL for key when iterating over images

Why?

In some cases the same image is used at different breakpoints meaning keys may not be unique when iterating through the images in ResponsiveImage:

447928498-bda9a94d-3853-4d3b-8696-e8307ab1601a

@jamesmockett jamesmockett requested a review from a team as a code owner January 14, 2026 15:37
@jamesmockett jamesmockett self-assigned this Jan 14, 2026
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@jamesmockett jamesmockett added the fix Departmental tracking: fix label Jan 14, 2026
@jamesmockett jamesmockett linked an issue Jan 14, 2026 that may be closed by this pull request
@jamesmockett jamesmockett added the run_chromatic Runs chromatic when label is applied label Jan 14, 2026
@jamesmockett jamesmockett force-pushed the jm/designable-banner-dup-key branch from 04b48c0 to 030c94e Compare January 14, 2026 15:38
@github-actions
Copy link

github-actions bot commented Jan 14, 2026

@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 14, 2026
@github-actions
Copy link

github-actions bot commented Jan 14, 2026

@shesah shesah added this to the Health milestone Jan 15, 2026
@jamesmockett jamesmockett force-pushed the jm/designable-banner-dup-key branch from 030c94e to c8aabb3 Compare January 15, 2026 12:34
@jamesmockett jamesmockett added the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@jamesmockett jamesmockett merged commit 4d4322c into main Jan 15, 2026
31 checks passed
@jamesmockett jamesmockett deleted the jm/designable-banner-dup-key branch January 15, 2026 12:41
@gu-prout
Copy link

gu-prout bot commented Jan 15, 2026

Seen on PROD (merged by @jamesmockett 7 minutes and 8 seconds ago) Please check your changes!

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

Labels

fix Departmental tracking: fix Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

React errors: duplicate key in Designable Banner

3 participants