Skip to content

Commit

Permalink
Merge pull request #349 from scientist-softserv/update-charts-for-gtm
Browse files Browse the repository at this point in the history
Update charts for google tag manager
  • Loading branch information
alishaevn authored Jan 24, 2024
2 parents a44a17f + b60eda2 commit 73a98d5
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 5 deletions.
3 changes: 0 additions & 3 deletions .env
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,3 @@ NEXT_PUBLIC_PROVIDER_ID=5159
NEXT_PUBLIC_SCIENTIST_API_VERSION=v2
NEXT_PUBLIC_WEBHOOK_URL=http://ss-mailer/webstore
NEXT_PUBLIC_APP_BASE_URL=https://webstore-staging.vercel.app

# optional/conditional variables
GOOGLE_TAG_MANAGER_ID=GTM-PQGCP24G
2 changes: 2 additions & 0 deletions charts/webstore/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,8 @@ spec:
value: "{{ .Values.nextAuthUrl }}"
- name: NEXT_PUBLIC_APP_BASE_URL
value: "{{ .Values.appBaseUrl }}"
- name: NEXT_PUBLIC_GOOGLE_TAG_MANAGER_ID
value: "{{ .Values.googleTagManagerId }}"
- name: NEXT_PUBLIC_PROVIDER_ID
value: "{{ .Values.providerId }}"
- name: NEXT_PUBLIC_PROVIDER_NAME
Expand Down
1 change: 1 addition & 0 deletions charts/webstore/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ appBaseUrl: ''
clientId: ''
clientSecret: ''
fullnameOverride: ''
googleTagManagerId: ''
imagePullSecrets: []
nameOverride: ''
nextAuthSecret: ''
Expand Down
1 change: 1 addition & 0 deletions ops/production-deploy.tmpl.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
appBaseUrl: 'https://webstore.softserv.cloud'
clientId: $CLIENT_ID
clientSecret: $CLIENT_SECRET
googleTagManagerId: ''
nextAuthSecret: $NEXTAUTH_SECRET
nextAuthUrl: 'https://webstore.softserv.cloud/api/auth'
nextPublicToken: $NEXT_PUBLIC_TOKEN
Expand Down
1 change: 1 addition & 0 deletions ops/staging-deploy.tmpl.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
appBaseUrl: 'https://webstore-staging.softserv.cloud'
clientId: $CLIENT_ID
clientSecret: $CLIENT_SECRET
googleTagManagerId: ''
nextAuthSecret: $NEXTAUTH_SECRET
nextAuthUrl: 'https://webstore-staging.softserv.cloud/api/auth'
nextPublicToken: $NEXT_PUBLIC_TOKEN
Expand Down
3 changes: 1 addition & 2 deletions pages/_app.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ import {
headerAndFooterLinkColors,
} from '../utils'
import '../utils/theme/globals.scss'
const gtmId = process.env.NODE_ENV === 'production' ? process.env.GOOGLE_TAG_MANAGER_ID : ''

const WebStore = ({ Component }) => {
/**
Expand All @@ -42,7 +41,7 @@ const WebStore = ({ Component }) => {
enableCookies={enableCookies}
getCookieConsent={getCookieConsent()}
/> */}
<GoogleTagManager gtmId={gtmId} />
<GoogleTagManager gtmId={process.env.NEXT_PUBLIC_GOOGLE_TAG_MANAGER_ID} />
<Header
auth={{
signIn: () => signIn(process.env.NEXT_PUBLIC_PROVIDER_NAME),
Expand Down

1 comment on commit 73a98d5

@vercel
Copy link

@vercel vercel bot commented on 73a98d5 Jan 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.