Skip to content

Conversation

@tomdavies73
Copy link
Contributor

Proposed behaviour

Updates the usage docs to remove any mentions of prop types and offer direct guidance regarding component imports.

Current behaviour

N/A

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • All themes are supported if required
  • Unit tests added or updated if required
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required
  • Translations added or updated (including creating or amending translation keys table in storybook) if required
  • Typescript d.ts file added or updated if required
  • Related docs have been updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

Copy link
Contributor

@edleeks87 edleeks87 left a comment

Choose a reason for hiding this comment

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

Thanks for raising this @tomdavies73 definitely fills a gap in our docs

@DipperTheDan DipperTheDan self-requested a review January 26, 2026 13:50
Copy link
Contributor

@DipperTheDan DipperTheDan left a comment

Choose a reason for hiding this comment

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

Nothing more to add from me. Just address all the points made by @edleeks87.

@tomdavies73 tomdavies73 marked this pull request as ready for review January 27, 2026 10:38
@tomdavies73 tomdavies73 requested a review from a team as a code owner January 27, 2026 10:38
@tomdavies73 tomdavies73 merged commit 803bc76 into master Jan 27, 2026
29 checks passed
@tomdavies73 tomdavies73 deleted the update_usage_docs branch January 27, 2026 10:57
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