-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Create Block: Enhancements to Gutenpride tutorial template #28215
Conversation
Size Change: 0 B Total Size: 1.28 MB ℹ️ View Unchanged
|
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.
Looks great, @gziolo! I'll be sure to reflect these changes in my suggested edits to the tutorial. I was working on it today and noticed some of the inconsistencies that you fixed here. Thanks! 🤩
@gziolo, @fabiankaegy I tested the template and it looks good. No errors. Everything is working as it should. |
* Create Block: Enhancements to the tutorial template * Apply coding style changes and align comments with the default template * Ensure that all validations pass for scaffolded block
Description
Follow-up for #27881.
This PR fixes linting errors reported in the scaffolded block with:
wp-scripts lint-js
andwp-scripts lint-js
In addition to that, it brings all the latest fixes applied to the
esnext
template.I also discovered that the comment that lists allowed categories wasn't updated in templates.
Finally, I added the
example
for the block to have a nice preview on the front-end.How has this been tested?
npx wp-create-block --template @wordpress/create-block-tutorial-template --no-wp-scripts cd gutenpride ../node_modules/.bin/wp-scripts format-js ../node_modules/.bin/wp-scripts lint-js ../node_modules/.bin/wp-scripts lint-style ../node_modules/.bin/wp-scripts build
Observe that the block is scaffolded, there are no errors reported and you are able to activate the block (plugin) in the editor.
Screenshots
It works for me:
Types of changes
Checklist: