Skip to content

Commit c86e68e

Browse files
committed
fix: solve new relic issue for build on pipeline
1 parent a17aaac commit c86e68e

7 files changed

Lines changed: 23 additions & 10 deletions

File tree

.circleci/config.yml

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,15 @@ jobs:
5555

5656
build:
5757
docker:
58-
- image: cimg/node:18.12.1
58+
- image: cimg/node:18.14.2
5959
resource_class: medium
6060
environment:
6161
FOLDER: checkout
6262
NEXT_PUBLIC_DOMAIN: commercelayer.co
6363
NEXT_PUBLIC_HOSTED: true
64+
NEXT_PUBLIC_SLUG: ""
65+
NEXT_PUBLIC_NEWRELIC_CONFIG_LOADER: ""
66+
NODE_ENV: production
6467
NEXT_PUBLIC_BASE_PATH: /checkout
6568
steps:
6669
- checkout
@@ -78,15 +81,23 @@ jobs:
7881

7982
build-on-pci-bucket:
8083
docker:
81-
- image: cimg/node:18.12.1
84+
- image: cimg/node:18.14.2
8285
resource_class: medium
8386
environment:
8487
FOLDER: checkout
8588
NEXT_PUBLIC_DOMAIN: commercelayer.io
89+
NEXT_PUBLIC_SLUG: ""
90+
NODE_ENV: production
8691
NEXT_PUBLIC_HOSTED: true
8792
NEXT_PUBLIC_BASE_PATH: /checkout
8893
steps:
8994
- checkout
95+
- run:
96+
name: Get version
97+
command: |
98+
pnpm -v
99+
npm -v
100+
node -v
90101
- run:
91102
name: Install
92103
command: pnpm install
@@ -144,4 +155,4 @@ workflows:
144155
tags:
145156
only: /^v(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*).*/
146157
branches:
147-
only: pci
158+
only: main

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,5 @@ cypress/fixtures/*
3737

3838
# vercel
3939
.vercel
40+
41+
Dockerfile.production

components/data/NewRelicSnippet/index.tsx

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@
9292
"i18next": "^21.10.0",
9393
"jwt-decode": "^3.1.2",
9494
"next": "^12.3.4",
95+
"next-build-id": "^3.0.0",
9596
"next-i18next": "^12.1.0",
9697
"postcss": "^8.4.21",
9798
"react": "^18.2.0",
@@ -124,7 +125,6 @@
124125
"eslint-plugin-promise": "^6.1.1",
125126
"eslint-plugin-react": "^7.32.2",
126127
"eslint-plugin-tailwind": "^0.2.1",
127-
"next-build-id": "^3.0.0",
128128
"prettier": "^2.8.3",
129129
"serve": "^14.1.2",
130130
"stylelint": "^14.16.1",

pages/_document.page.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class AppDocument extends Document {
2525
rel="stylesheet"
2626
/>
2727

28-
{process.env.NEXT_PUBLIC_NEWRELIC_LOADER_CONFIG && (
28+
{process.env.NEXT_PUBLIC_NEWRELIC_LOADER_CONFIG !== null && (
2929
<Script id="new-relic" strategy="afterInteractive">
3030
{NewRelicSnippet}
3131
</Script>

pnpm-lock.yaml

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

utils/getSettings.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,14 +162,14 @@ export const getSettings = async ({
162162
}
163163

164164
if (isProduction() && (subdomain !== slug || kind !== "sales_channel")) {
165-
// return invalidateCheckout()
165+
return invalidateCheckout()
166166
} else if (kind !== "sales_channel") {
167167
return invalidateCheckout()
168168
}
169169

170170
const cl = CommerceLayer({
171171
organization: slug,
172-
accessToken: accessToken,
172+
accessToken,
173173
domain,
174174
})
175175

0 commit comments

Comments
 (0)