Skip to content

Conversation

andreaskienle
Copy link
Contributor

This PR fixes the console error with duplicate keys in the breadcrumbs.
image

Since paths aren’t unique (yet!), I’ve switched the key from path to index. This should be fine for now, as the items are static.

@Copilot Copilot AI review requested due to automatic review settings October 2, 2025 10:39
Copy link
Contributor

@Copilot 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 a React console error caused by duplicate keys in the breadcrumbs component by switching from using path as the key to using the array index.

  • Replaces item.path with index as the React key prop to resolve duplicate key warnings
  • Maintains the data-target attribute using the original item.path value

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@Hubert-Szczepanski-SAP Hubert-Szczepanski-SAP left a comment

Choose a reason for hiding this comment

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

I checked on my side. Console error disappeared :)

@andreaskienle andreaskienle merged commit caabac3 into main Oct 2, 2025
5 checks passed
@andreaskienle andreaskienle deleted the fix/fix-breadcrumbs-key branch October 2, 2025 10:54
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.

2 participants