fix: dont use window for component scope

newrelic · Jul 8, 2020 · ffed14c · ffed14c
1 parent 2fb389d
commit ffed14c
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/src/components/ReferenceExample.js b/src/components/ReferenceExample.js
@@ -1,4 +1,4 @@
-import React, { useCallback } from 'react';
+import React, { useCallback, useMemo } from 'react';
 import PropTypes from 'prop-types';
 import styles from './ReferenceExample.module.scss';
 import ReferencePreview from './ReferencePreview';
@@ -16,13 +16,6 @@ const nerdletStateContextMock = {
   entityGuid: 'MTIzNDU2fEZPT3xCQVJ8OTg3NjU0MzIx',
 };
 
-const SCOPE = {
-  ...window.__NR1_SDK__.default,
-  navigation: {
-    getOpenLauncherLocation: () => {},
-  },
-};
-
 const ReferenceExample = ({
   className,
   example,
@@ -35,6 +28,16 @@ const ReferenceExample = ({
   } = window.__NR1_SDK__.default;
   const { live } = example.options;
 
+  const scope = useMemo(
+    () => ({
+      ...window.__NR1_SDK__.default,
+      navigation: {
+        getOpenLauncherLocation: () => {},
+      },
+    }),
+    []
+  );
+
   const Preview = useCallback(
     ({ className }) => (
       <ReferencePreview
@@ -56,7 +59,7 @@ const ReferenceExample = ({
             language="jsx"
             live={live}
             preview={live}
-            scope={SCOPE}
+            scope={scope}
             components={{ Preview }}
           >
             {example.sourceCode}