-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Linking between pages doc #8626
Conversation
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.
I think having this doc is very helpful -- not everyone is going to go through the tutorial. I have the same comment i've had recently though, which is that I'm not sure how helpful it is to include the entire code example from the tutorial -- it strikes me as unnecessarily duplicative.
I'd suggest:
- Intro language (which you have)
- A basic code block showing how to use the Link component (but not in a "step-by-step" guided kind of way.
- a link out to the full step-by-step example in the tutorial
Did you mean to link to the API docs twice? I'm not sure it's needed in the body of the doc, and at the end, but that's up to you! Lmk when you think.
How's this, @amberleyromo ?
@shannonbux I like the changes -- good to go, or still tweaking? |
Nevermind, saw your comment in the commit a99974a I think this is good, quick example and linking out to the longer tutorial explanation 👍 |
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.
👍
Awesome thanks @amberleyromo!
…On Mon, Oct 1, 2018 at 1:53 PM Amberley ***@***.***> wrote:
Merged #8626 <#8626> into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#8626 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/Ae9o2rb1qTYiMiJHNhiPxxk8CTZf9tgpks5ugnLPgaJpZM4W_Z8i>
.
|
Wondering if the Gatsby Link doc (docs/gatsby-link/) is actually all we need? Not sure...