add app insights repo secret usage

microsoft · Aug 9, 2023 · c1fd841 · c1fd841
1 parent fe2371b
commit c1fd841
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/docsite-publish-ghpages.yml b/.github/workflows/docsite-publish-ghpages.yml
@@ -51,6 +51,8 @@ jobs:
 
       - name: Build storybook
         run: yarn workspace @fluentui/public-docsite-v9 build-storybook
+        env:
+          STORYBOOK_APPINSIGHTS_INSTRUMENTATION_KEY: ${{ secrets.STORYBOOK_APPINSIGHTS_INSTRUMENTATION_KEY }}
 
       - name: Upload Pages Artifact
         uses: actions/upload-pages-artifact@v1

diff --git a/apps/public-docsite-v9/.storybook/manager.js b/apps/public-docsite-v9/.storybook/manager.js
@@ -10,12 +10,12 @@ addons.setConfig({
 
 addons.register('application-insights', api => {
   if (process.env.NODE_ENV === 'production') {
-    const { STORYBOOK_INSTRUMENTATION_KEY } = process.env;
+    const { STORYBOOK_APPINSIGHTS_INSTRUMENTATION_KEY } = process.env;
 
-    if (STORYBOOK_INSTRUMENTATION_KEY) {
+    if (STORYBOOK_APPINSIGHTS_INSTRUMENTATION_KEY) {
       const appInsights = new ApplicationInsights({
         config: {
-          connectionString: STORYBOOK_INSTRUMENTATION_KEY,
+          connectionString: STORYBOOK_APPINSIGHTS_INSTRUMENTATION_KEY,
           disableCookiesUsage: true,
         },
       });
@@ -35,7 +35,7 @@ addons.register('application-insights', api => {
       api.on(STORY_ERRORED, trackError);
       api.on(STORY_MISSING, trackError);
     } else {
-      console.warn(`[application-insights] instrumentation key not found`);
+      console.warn(`[application-insights] instrumentation key not found in window or env variable`);
     }
   }
 });