Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: Apollo-Expose-Query-Plan header #7147

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

docs: Apollo-Expose-Query-Plan header #7147

wants to merge 1 commit into from

Conversation

shorgi
Copy link
Contributor

@shorgi shorgi commented Mar 28, 2025

Update Apollo-Expose-Query-Plan header doc

@shorgi shorgi requested a review from a team as a code owner March 28, 2025 22:34
@svc-apollo-docs
Copy link
Collaborator

svc-apollo-docs commented Mar 28, 2025 •
edited
Loading

✅ Docs preview ready

The preview is ready to be viewed. View the preview

File Changes

0 new, 1 changed, 0 removed 
* graphos/routing/(latest)/observability/debugging-subgraph-requests.mdx

Build ID: a41d7ba7c8210e393e33ae9b

URL: https://www.apollographql.com/docs/deploy-preview/a41d7ba7c8210e393e33ae9b

@shorgi shorgi requested a review from dariuszkuc March 28, 2025 22:34
@github-actions GitHub Actions
Copy link
Contributor

@shorgi, please consider creating a changeset entry in /.changesets/. These instructions describe the process and tooling.

@shorgi shorgi added the backport-1.x Backport this PR to 1.x label Mar 28, 2025
Meschreiber
Meschreiber reviewed Mar 29, 2025
View reviewed changes
Copy link
Contributor

@Meschreiber Meschreiber left a comment

Choose a reason for hiding this comment

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

@shorgi , do you think this is a better or just additional spot for this info? (also in #7143)

@dariuszkuc
Copy link
Member

Looks like this was updated in the federations docs apollographql/federation#3226 instead of the router docs. Can you update this PR to match the federation one (as it is more accurate)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Meschreiber Meschreiber Meschreiber left review comments

@dariuszkuc dariuszkuc Awaiting requested review from dariuszkuc

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
backport-1.x Backport this PR to 1.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@shorgi @svc-apollo-docs @dariuszkuc @Meschreiber