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.

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

Delete src/pages/graphql-js directory #1945

Open
wants to merge 5 commits into
base: source
Choose a base branch
from

Conversation

benjie
Copy link
Member

@benjie benjie commented Jan 23, 2025

Closes #1943

I've not tested this locally, plan to rely on Vercel's preview to check all is well.

Copy link

vercel bot commented Jan 23, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
graphql-github-io ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 4, 2025 11:13am

@benjie
Copy link
Member Author

benjie commented Jan 23, 2025

This is insufficient; some of the internal links don't work now e.g. "GraphQL.js Tutorial" in the header.

@benjie
Copy link
Member Author

benjie commented Feb 4, 2025

That link has now been deleted it seems 🤷‍♂️

@benjie benjie marked this pull request as ready for review February 4, 2025 11:06
@benjie
Copy link
Member Author

benjie commented Feb 4, 2025

I've not had time to properly go through and check everything in this, but I think it's at least better than the status quo.

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.

GraphQL-js documentation should be deleted once new graphql-js.org redirects have proved stable
2 participants