Skip to content

Conversation

sarahxsanders
Copy link
Contributor

This PR has some cleanup tasks:

  • Reorganized the information architecture a bit so that sections flow better
  • Fixed broken links
  • Fixed a code snippet that wasn't closed off/bleeding into a section
  • Updated auth strategy guide w/ callouts for not using resolver auth in production per @benjie request

Please let me know if there are any other tweaks I can include!

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.

Excellent; thanks Sarah!

@JoviDeCroock JoviDeCroock merged commit 9874ed6 into graphql:16.x.x May 30, 2025
20 checks passed
yaacovCR pushed a commit to yaacovCR/graphql-js that referenced this pull request May 30, 2025
This PR has some cleanup tasks:
- Reorganized the information architecture a bit so that sections flow
better
- Fixed broken links
- Fixed a code snippet that wasn't closed off/bleeding into a section
- Updated auth strategy guide w/ callouts for not using resolver auth in
production per @benjie request

Please let me know if there are any other tweaks I can include!

---------

Co-authored-by: Benjie <[email protected]>
Co-authored-by: Jovi De Croock <[email protected]>
yaacovCR pushed a commit that referenced this pull request May 30, 2025
This PR has some cleanup tasks:
- Reorganized the information architecture a bit so that sections flow
better
- Fixed broken links
- Fixed a code snippet that wasn't closed off/bleeding into a section
- Updated auth strategy guide w/ callouts for not using resolver auth in
production per @benjie request

Please let me know if there are any other tweaks I can include!

---------

Co-authored-by: Benjie <[email protected]>
Co-authored-by: Jovi De Croock <[email protected]>
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.

3 participants