Merge pull request #90 from WordPress/fix-context-not-updated-on-prop…

…-change 🎨 Fix context not updated on prop change
WordPress · Oct 21, 2022 · 0f90f67 · 0f90f67
2 parents 647589d + 8e11605
commit 0f90f67
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/runtime/components.js b/src/runtime/components.js
@@ -4,7 +4,7 @@ import { component } from './hooks';
 
 export default () => {
 	const WpContext = ({ children, data, context: { Provider } }) => {
-		const signals = useMemo(() => deepSignal(JSON.parse(data)), []);
+		const signals = useMemo(() => deepSignal(JSON.parse(data)), [data]);
 		return <Provider value={signals}>{children}</Provider>;
 	};
 	component('wp-context', WpContext);

diff --git a/src/runtime/directives.js b/src/runtime/directives.js
@@ -17,11 +17,13 @@ export default () => {
 	directive(
 		'context',
 		({
-			directives: { context },
+			directives: {
+				context: { default: context },
+			},
 			props: { children },
 			context: { Provider },
 		}) => {
-			const signals = useMemo(() => deepSignal(context.default), []);
+			const signals = useMemo(() => deepSignal(context), [context]);
 			return <Provider value={signals}>{children}</Provider>;
 		}
 	);