Skip to content

GQL Playground: Only run in dev mode#1655

Merged
Tobbe merged 2 commits intoredwoodjs:mainfrom
Tobbe:tobbe-gql-playground-dev-only
Jan 18, 2021
Merged

GQL Playground: Only run in dev mode#1655
Tobbe merged 2 commits intoredwoodjs:mainfrom
Tobbe:tobbe-gql-playground-dev-only

Conversation

@Tobbe
Copy link
Copy Markdown
Contributor

@Tobbe Tobbe commented Jan 18, 2021

Fixes #1591

...options
}: GraphQLHandlerOptions = {}) => {
const isDevEnv = process.env.NODE_ENV !== 'production'
const isDevEnv = process.env.NODE_ENV === 'development'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I thought about adding || process.env.NODE_ENV === 'test' as well, but the playground isn't needed when running tests, right? And neither is introspection I guess. Maybe we want debug. What do you all think?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Agreed!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 18, 2021

📦 PR Packages

Click to Show Package Download Links

https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/create-redwood-app-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-api-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-api-server-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-auth-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-cli-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-core-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-dev-server-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-eslint-config-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-eslint-plugin-redwood-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-forms-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-internal-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-router-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-structure-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-testing-0.23.0-19c07fb.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1655/redwoodjs-web-0.23.0-19c07fb.tgz

Install this PR by running yarn rw upgrade --pr 1655:0.23.0-19c07fb

@peterp peterp added this to the next release milestone Jan 18, 2021
Copy link
Copy Markdown
Member

@peterp peterp left a comment

Choose a reason for hiding this comment

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

🔥 nice job Tobbe!

@dac09
Copy link
Copy Markdown
Contributor

dac09 commented Jan 18, 2021

This is great! 🎉thanks @Tobbe

@Tobbe Tobbe merged commit 3686353 into redwoodjs:main Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ApolloServer GraphQL Playground not necessarily turned off in production

3 participants