-
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
Add Post Comment block #24781
Add Post Comment block #24781
Conversation
61ff5b8
to
8be0a42
Compare
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.
Unit tests are failing because of missing fixtures for the two new blocks that this PR introduces. This doc has some information on how to create them 🙂
…bo97/gutenberg into add/post-comment-block
@ockham Gotcha! Added the icons and fixtures. Tests are happy now! |
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.
Looking good! This will provide us with a good baseline to work on other comment related blocks.
Description
Related: #24101
Laying the foundation for a block-based post comment building. Adds a Post Comment block which requires a Comment ID input for now. The Post Comment block provides a context for the inner blocks.
As an example, also implemented a very simple Post Comment Content block, which uses the context provided by the Post Comment block.
How has this been tested?
Types of changes
New feature
Checklist: