-
Notifications
You must be signed in to change notification settings - Fork 135
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
Bug: Fixing null/undefined theme argument for the WordPressBlocksProvider #2013
base: canary
Are you sure you want to change the base?
Conversation
…PressBlocksProvider
|
📦 Next.js Bundle Analysis for @faustwp/getting-started-exampleThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
@moonmeister Can you review this please and see if you can still replicate issue #1986 please |
@moonmeister I need to update this PR as some of the logic isn't correct |
@moonmeister This is ready for review. |
Can you explain why/how this fixes the issue? |
Tasks
Description
Fixing an issue with the WordPressBlocksProvider that doesn't allow the theme argument to be null.
Related Issue(s):
#1986
Testing
I have tested this on 2 different ways
Screenshots
Documentation Changes
Dependant PRs