Skip to content

Add documentation for the search tool #201

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
Merged
merged 4 commits into from
Jul 31, 2025
Merged

Add documentation for the search tool #201

merged 4 commits into from
Jul 31, 2025

Conversation

pubmodmatt
Copy link

@pubmodmatt pubmodmatt commented Jul 16, 2025 •
edited
Loading

See the deploy preview here.

Draft until a release is available that includes the search tool.

@pubmodmatt pubmodmatt self-assigned this Jul 16, 2025
@apollo-librarian
Copy link

apollo-librarian bot commented Jul 16, 2025 •
edited
Loading

✅ Docs preview ready

The preview is ready to be viewed. View the preview

File Changes

0 new, 3 changed, 0 removed 
* (developer-tools)/apollo-mcp-server/(latest)/best-practices.mdx
* (developer-tools)/apollo-mcp-server/(latest)/quickstart.mdx
* (developer-tools)/apollo-mcp-server/(latest)/guides/index.mdx

Build ID: aaa82eacb1b1beb6693c169a
Build Logs: View logs

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

DaleSeo
DaleSeo reviewed Jul 16, 2025
View reviewed changes
Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

Should we also add a configuration guide somewhere?

@pubmodmatt
Copy link
Author

Should we also add a configuration guide somewhere?

I've explicitly not addressed configuration here, as almost all of the configuration documentation will need to be updated due to the move from CLI arguments to config file.

riginoommen
riginoommen reviewed Jul 17, 2025
View reviewed changes
DaleSeo
DaleSeo approved these changes Jul 17, 2025
View reviewed changes
Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for adding the docs.

@DaleSeo DaleSeo force-pushed the pubmodmatt/search_doc branch from 6c1f863 to 50640d2 Compare July 31, 2025 19:30
@DaleSeo DaleSeo marked this pull request as ready for review July 31, 2025 19:32
@DaleSeo DaleSeo requested review from a team as code owners July 31, 2025 19:32
@DaleSeo DaleSeo assigned DaleSeo and unassigned pubmodmatt Jul 31, 2025
mabuyo
mabuyo approved these changes Jul 31, 2025
View reviewed changes
Copy link
Contributor

@mabuyo mabuyo left a comment

Choose a reason for hiding this comment

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

Minor nits!

DaleSeo and others added 2 commits July 31, 2025 16:54
@DaleSeo DaleSeo merged commit b750551 into main Jul 31, 2025
7 checks passed
@DaleSeo DaleSeo deleted the pubmodmatt/search_doc branch August 25, 2025 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DaleSeo DaleSeo DaleSeo approved these changes

@mabuyo mabuyo mabuyo approved these changes

+1 more reviewer

@riginoommen riginoommen riginoommen left review comments

Reviewers whose approvals may not affect merge requirements
Assignees

@DaleSeo DaleSeo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pubmodmatt @riginoommen @DaleSeo @mabuyo