Skip to content

Bug: Fixing null/undefined theme argument for the WordPressBlocksProvider #2013

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.

Sign up for GitHub

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

Merged
josephfusco merged 14 commits into from
Jan 8, 2025
Merged

Bug: Fixing null/undefined theme argument for the WordPressBlocksProvider #2013

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
fb7f89c
Fixes #1986 and allows for the theme argument to be null for the Word…
colinmurphy Dec 20, 2024
4514aed
Fixed typo
colinmurphy Dec 20, 2024
1e6fd19
Fixing lint issue if theme context is undefined.
colinmurphy Dec 20, 2024
24f4d0e
Fixed conditions to allow null for the theme argument
colinmurphy Dec 20, 2024
54cfbaf
Fixed logic
colinmurphy Dec 20, 2024
d7146e3
Fixed lint issues
colinmurphy Dec 20, 2024
73f8a2f
Fixed typo after running eslint and it changed WordPressThemeContext
colinmurphy Dec 23, 2024
0493012
Returning theme context as this isn't required by the WordPressBlockP…
colinmurphy Dec 24, 2024
b94fd14
Set return type as BlocksTheme
colinmurphy Dec 24, 2024
0d65fa3
Removed test assertion to check if theme is undefined
colinmurphy Dec 24, 2024
1d1f608
Updating to use a default value for theme instead in order to prevent…
colinmurphy Jan 3, 2025
c53812b
Removed testing code
colinmurphy Jan 3, 2025
c6fcb91
Reverted test deletion.
colinmurphy Jan 8, 2025
9059522
Addec changeset
colinmurphy Jan 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/afraid-terms-shake.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@faustwp/blocks": patch
---

Bug: Fixed an issue an issue with WordPressBlocksProvider and the theme argument to allow it to be optional and not throw an error. By default theme is now an empty object
2 changes: 1 addition & 1 deletion packages/blocks/src/components/WordPressBlocksProvider.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export function WordPressBlocksProvider(props: {
config: WordPressBlocksProviderConfig;
}) {
const { children, config } = props;
const { blocks, theme } = config;
const { blocks, theme = {} } = config;

return (
<WordPressBlocksContext.Provider value={blocks}>
Expand Down
Loading