Merge pull request #390 from scientist-softserv/rename-webhook

rename-webhook
scientist-softserv · Mar 4, 2024 · c21860d · c21860d
2 parents 2c1650b + 261ff04
commit c21860d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/.env.example b/.env.example
@@ -1,6 +1,8 @@
 # reference https://nextjs.org/docs/basic-features/environment-variables#exposing-environment-variables-to-the-browser
 # to learn about the NEXT_PUBLIC prefix
 
+# Ref: https://github.com/scientist-softserv/webstore/wiki/Deployment#when-using-a-cname
+# when deploying to a custom domain.
 NEXT_PUBLIC_APP_BASE_URL=https://store.webstore.com
 NEXT_PUBLIC_PROVIDER_ID=0001
 NEXT_PUBLIC_PROVIDER_NAME=webstore

diff --git a/pages/_app.js b/pages/_app.js
@@ -21,6 +21,7 @@ import {
   headerAndFooterLinkColors,
 } from '../utils'
 import '../utils/theme/globals.scss'
+const gtmId = process.env.NEXT_PUBLIC_GOOGLE_TAG_MANAGER_ID
 
 const WebStore = ({ Component }) => {
   /**
@@ -41,7 +42,7 @@ const WebStore = ({ Component }) => {
         enableCookies={enableCookies}
         getCookieConsent={getCookieConsent()}
       /> */}
-      <GoogleTagManager gtmId={process.env.NEXT_PUBLIC_GOOGLE_TAG_MANAGER_ID} />
+      {gtmId && <GoogleTagManager gtmId={gtmId} />}
       <Header
         auth={{
           signIn: () => signIn(process.env.NEXT_PUBLIC_PROVIDER_NAME),

diff --git a/utils/api/webhooks.js b/utils/api/webhooks.js
@@ -11,7 +11,7 @@ export const getWebhookConfig = async (accessToken) => {
 export const createWebhookConfig = (accessToken) => {
   /* eslint-disable camelcase */
   const webhook_config = {
-    'name': 'Webstore',
+    'name': `${process.env.NEXT_PUBLIC_PROVIDER_NAME} DS`,
     'url': `${process.env.NEXT_PUBLIC_WEBHOOK_URL}`,
     'active': true,
     'params': {