-
Notifications
You must be signed in to change notification settings - Fork 277
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Throw error when run jss start using FETCH_WITH=GraphQL #920
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/sitecore-jss/edge-e2e-styleguide/5k6ruPPVm5trTfPLdwsS2RTSVogZ [Deployment for be8d3d8 failed] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just some suggestions
packages/create-sitecore-jss/src/templates/angular/scripts/bootstrap.ts
Outdated
Show resolved
Hide resolved
packages/create-sitecore-jss/src/templates/nextjs/scripts/bootstrap.ts
Outdated
Show resolved
Hide resolved
packages/create-sitecore-jss/src/templates/react/scripts/bootstrap.js
Outdated
Show resolved
Hide resolved
packages/create-sitecore-jss/src/templates/vue/scripts/bootstrap.js
Outdated
Show resolved
Hide resolved
packages/create-sitecore-jss/src/templates/angular/package.json
Outdated
Show resolved
Hide resolved
…pt in package.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One question, otherwise looks great!
@@ -23,6 +23,7 @@ | |||
"allowSyntheticDefaultImports": true, | |||
"esModuleInterop": true, | |||
"resolveJsonModule": true, | |||
"useUnknownInCatchVariables": false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious - what was the reason for adding this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm. This change actually seems to be breaking the build on CI.
tsconfig.json(26,5): error TS5023: Unknown compiler option 'useUnknownInCatchVariables'.
We should probably revert this. Guessing it's something on your local env that's causing the error for you?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, probably something with my local environment.
The user should see a friendly error message when they try to run jss start i.e disconnected mode when fetchWtih is set to 'GraphQL'.
Description / Motivation
Testing Details
Types of changes