-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Update dynamic blocks documentation #16450
Conversation
Added in adjustments that @chrisvanpatten recommended on the last update, but they were accidentally removed before the final pull request. Also have added in a line mentioning that if using InnerBlocks these need to be saved using the save callback.
One of the tests failed, but seems like a mistake, as no code was actually changed on this file - just text changes. Can someone look into this please? |
@jodamo5 We had some issues with the tests yesterday. I restarted them and now they are fine :-) |
Thanks @Soean. Tests have passed. Can someone please merge this in? Thanks. |
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.
I added a suggestion to show how to save the content. If there is a link to an example block that would be good to include but not necessary.
docs/designers-developers/developers/tutorials/block-tutorial/creating-dynamic-blocks.md
Outdated
Show resolved
Hide resolved
…creating-dynamic-blocks.md Example added showing how to save InnerBlocks. Co-Authored-By: Marcus Kazmierczak <[email protected]>
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 good, thanks for the updates!
#16374 Description
Added in adjustments that @chrisvanpatten recommended on the last update, but they were accidentally removed before the final pull request. Also have added in a line mentioning that if using InnerBlocks these need to be saved using the save callback.
How has this been tested?
Most changes here are simply text changes to improve the understanding. The InnerBlocks explanation has been tested on both development and live sites.
Types of changes
Just further explanations in the documentation.