Merge pull request #33 from ericz/patch-1

kentwalters · web-flow · commit 8bd05922fd12 · 2022-12-01T17:16:20.000-08:00
onData is supposed to be optional. Add optional chaining check
diff --git a/src/components/Retool.js b/src/components/Retool.js
@@ -31,7 +31,7 @@ const Retool = ({ data, url, height, width, onData }) => {
         event.data?.type !== "PARENT_WINDOW_QUERY" &&
         event.data?.type !== 'intercom-snippet__ready'
       ) {
-        onData(event.data);
+        onData?.(event.data);
       }
       
       if (event.data.type === "PARENT_WINDOW_QUERY") {

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ const Retool = ({ data, url, height, width, onData }) => {`
`31`	`31`	`event.data?.type !== "PARENT_WINDOW_QUERY" &&`
`32`	`32`	`event.data?.type !== 'intercom-snippet__ready'`
`33`	`33`	`) {`
`34`		`- onData(event.data);`
	`34`	`+ onData?.(event.data);`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`if (event.data.type === "PARENT_WINDOW_QUERY") {`