Skip to content

docs: update "going to production" guide #4398

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

Merged
merged 8 commits into from
May 29, 2025

Conversation

sarahxsanders
Copy link
Contributor

@sarahxsanders sarahxsanders commented May 15, 2025

Updates "Going to Production" guide to cover a larger scope. The goal is to cover most bases on what devs should consider before deploying to production. I also added a checklist.

I will preface that this is a bit of a long page, but useful to have

Not ready for review yet, still proofreading/cleaning up

@sarahxsanders sarahxsanders marked this pull request as ready for review May 16, 2025 20:45
@sarahxsanders sarahxsanders requested a review from a team as a code owner May 16, 2025 20:45
Copy link
Member

@benjie benjie left a comment

Choose a reason for hiding this comment

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

This is really good! I'm going to follow up with a commit to add trusted documents to the advice, users who use trusted documents can mostly skip over disabling introspection, query cost analysis, and similar concerns.

@benjie benjie merged commit 29324d5 into graphql:16.x.x May 29, 2025
20 checks passed
@benjie benjie mentioned this pull request May 29, 2025
@benjie
Copy link
Member

benjie commented May 29, 2025

Follow up with trusted documents can be found here: #4418

benjie added a commit that referenced this pull request May 29, 2025
Expands on #4398 by recommending the trusted documents pattern which
90+% of GraphQL users should be using.
yaacovCR pushed a commit to yaacovCR/graphql-js that referenced this pull request May 30, 2025
Updates "Going to Production" guide to cover a larger scope. The goal is
to cover most bases on what devs should consider before deploying to
production. I also added a checklist.

---------

Co-authored-by: Benjie <[email protected]>
yaacovCR pushed a commit to yaacovCR/graphql-js that referenced this pull request May 30, 2025
Expands on graphql#4398 by recommending the trusted documents pattern which
90+% of GraphQL users should be using.
yaacovCR pushed a commit that referenced this pull request May 30, 2025
Updates "Going to Production" guide to cover a larger scope. The goal is
to cover most bases on what devs should consider before deploying to
production. I also added a checklist.

---------

Co-authored-by: Benjie <[email protected]>
yaacovCR pushed a commit that referenced this pull request May 30, 2025
Expands on #4398 by recommending the trusted documents pattern which
90+% of GraphQL users should be using.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants