-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prepare for new block variations 1/x: hookable-patterns, slot-fills & docs #888
base: main
Are you sure you want to change the base?
Prepare for new block variations 1/x: hookable-patterns, slot-fills & docs #888
Conversation
…former source was deprecated in 6.6)
\register_block_pattern( | ||
'gatherpress/event-template', | ||
array( | ||
'title' => 'Invisible Event Template Block Pattern', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs internationalization __().
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a little funny.
- I had _() everywhere in place
- Than realised, that nobody will ever see this. Because it's hidden. So I removed the _() and added a comment "// Needs no i18n
- Lastly I removed the comment also, because .... I don't know.
Now I'm going to add _() again, yes.
* when registering the 'gatherpress_event' post type | ||
* and will not be visible to the editor at any point. | ||
*/ | ||
\register_block_pattern( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't need \
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be fixed, but I'm wondering why the CS checks didn't jump in here. Some days ago, I remember that phpcs was jelling at me because of a \WP_Query. But here, nothing. Hmmm.
Finished, but mention the two unresolved comments, please @mauteri . |
Preview changes with PlaygroundYou can preview the recent changes for PR#888 with the following PHP versions: PHP Version 8.3
PHP Version 7.4
Download Made with 💙 from GatherPress & a little bit of WordPress Playground. Changes will not persist between sessions. |
Preparation to bring some block variations from the gatherpress-block-playground into the plugin as part of #626.
Description of the Change
This PR does:
PluginDocumentSettingPanel
fromwordpress/editor
package (The former source was deprecated in 6.6, which led to the annoying console warnings all over the place)/docs/developer/blocks/...
Closes #628
How to test the Change
Changelog Entry
Credits
Props @carstingaxion
Checklist: