-
Notifications
You must be signed in to change notification settings - Fork 1
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 the WpBlock
custom element
#39
Conversation
Add cases for other source types.Add cases for other source types.
🚀 This comment was generated by the automations bot based on a
|
Size Change: 0 B Total Size: 878 kB ℹ️ View Unchanged
|
namespace JSX { | ||
interface IntrinsicElements { | ||
// eslint-disable-next-line @typescript-eslint/naming-convention | ||
'gutenberg-inner-blocks': React.DetailedHTMLProps< |
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.
We should probably also update the "nomenclature" here, see #21 (comment) 🙂
(tl;dr gutenberg
➡️ wp
, or wp-block
in other contexts)
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've taken the liberty of applying some nomenclature fixes in f7b465b. Feel free to revert if you don't like them! 😄
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.
Thanks @ockham ! I was going to do the same thing but you beat me to it :)
There are some action failures but they're not consequential:
|
This PR is adding the
<wp-block>
custom element to the frontend of woocommerce.The
<wp-block>
was originally created in the BHE repo. This PR is bringing it to WBHE.