Skip to content

Fix props in example code for Edit Post module#21976

Merged
gziolo merged 2 commits intoWordPress:masterfrom
ediamin:docs/fix-edit-post-doc-props
Apr 30, 2020
Merged

Fix props in example code for Edit Post module#21976
gziolo merged 2 commits intoWordPress:masterfrom
ediamin:docs/fix-edit-post-doc-props

Conversation

@ediamin
Copy link
Copy Markdown
Contributor

@ediamin ediamin commented Apr 29, 2020

Description

Fix the props in example code for Edit Post module's PluginBlockSettingsMenuItem API

How has this been tested?

I made a custom component that uses this example code snippet with my fixes.

Types of changes

Fixed with proper JSX syntax that we use to write props value

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.

@gziolo gziolo added [Package] Edit Post /packages/edit-post [Type] Developer Documentation Documentation for developers labels Apr 29, 2020
Copy link
Copy Markdown
Member

@gziolo gziolo 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 the fix 👍

Copy link
Copy Markdown
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

Travis fails and I discovered what’s the reason. The same change needs to be applied to JSDoc comment of the documented component. So it’s autogenerated from source code with npm run docs:build command.

@ediamin
Copy link
Copy Markdown
Contributor Author

ediamin commented Apr 30, 2020

@gziolo updated the JSDoc too. I hope this time Travis won't fail.

Copy link
Copy Markdown
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

All good now, thanks!

@gziolo gziolo merged commit 8efe9ef into WordPress:master Apr 30, 2020
@github-actions github-actions bot added this to the Gutenberg 8.1 milestone Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Package] Edit Post /packages/edit-post [Type] Developer Documentation Documentation for developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants