Skip to content

docs: correct type name in editMenuItems client component example#15916

Closed
ossaidqadri wants to merge 2 commits intopayloadcms:mainfrom
ossaidqadri:fix-edit-menu-items-type
Closed

docs: correct type name in editMenuItems client component example#15916
ossaidqadri wants to merge 2 commits intopayloadcms:mainfrom
ossaidqadri:fix-edit-menu-items-type

Conversation

@ossaidqadri
Copy link
Contributor

Description

This PR fixes an incorrect type name in the documentation for the editMenuItems client component example.

Changes

  • Changed EditViewMenuItemClientProps to EditMenuItemsClientProps in docs/custom-components/edit-view.mdx
  • Both the import statement and the prop type annotation were updated to use the correct type name

Motivation

The documentation was using an incorrect type name that doesn't match the actual exported type from the payload package. This would confuse users trying to implement custom edit menu items components.

Fixes #15898

Fix incorrect type name in docs/custom-components/edit-view.mdx
Changed EditViewMenuItemClientProps to EditMenuItemsClientProps to match
the actual exported type from the payload package.

Fixes payloadcms#15898
Copilot AI review requested due to automatic review settings March 11, 2026 20:01
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 a documentation typo in the editMenuItems client component example by updating the referenced prop type to match the type actually exported from payload.

Changes:

  • Updated the import in the client component example from EditViewMenuItemClientProps to EditMenuItemsClientProps.
  • Updated the example component’s prop type annotation to EditMenuItemsClientProps.

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

@GermanJablo
Copy link
Contributor

Closed in favor of #15904
I wanted to prioritize this one, but I don't know why it has a change unrelated to the issue outside of the documentation. Thanks for reporting the issue anyway!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change EditViewMenuItemClientProps to EditViewMenuItemsClientProps

3 participants