-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs] fix broken links on get started and guides #7992
Conversation
@@ -25,7 +25,7 @@ function githubUrl(path) { | |||
} else { | |||
path = path.replace('/versions/latest/', '/versions/unversioned/'); | |||
} | |||
} else if (path.match(/v\d+\.\d+\.\d+\/?$/)) { | |||
} else if (path.match(/v\d+\.\d+\.\d+\/?$/) || path === '/') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes the Edit on GitHub
from the intro page.
b1494c9
to
6e770a8
Compare
@@ -157,8 +157,6 @@ There are four major GraphQL clients available at the moment: | |||
|
|||
Apollo, Relay and Urql implement further features like caching, realtime support with GraphQL subscriptions or optimistic UI updates. | |||
|
|||
Learn how to integrate with Auth0 social providers in the [expo-auth0-example](https://github.com/graphcool-examples/react-native-graphql/tree/master/authentication-with-expo-and-auth0) repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now outdated and not supported, graphcool also doesn't exist anymore (now prisma). 馃槄 This page could use a refresh
Why
I ran a full check of all links, there is still a lot left in the API reference. But this fixes the "Get Started" and "Guides" sections. (Fixes #7985)
How
Ran export locally and double-checked the links.
Test Plan
Could be great if we can detect internal links and/or headers by ESLint or something related 馃槃