-
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
[Tutorial] Add notice regarding 4-part tutorial near the top #5673
Conversation
Deploy preview for using-drupal ready! Built with commit 9fa938b |
Deploy preview for gatsbygram ready! Built with commit 9fa938b |
Thanks @ryanditjia. This looks good to me. cc @shannonbux for another set of eyes on this. Preview links are: https://deploy-preview-5673--gatsbyjs.netlify.com/tutorial/part-four/ Edit: pressed enter too early |
Coming back to this - the note feels a bit too prominent to me. One idea on condensing it would be:
Something like: This tutorial is part of a series about Gatsby's data layer, make sure you've gone through part X before continuing here. On parts 6 and 7 this could be extended with Are you new here? Jump back to the start of this tutorial series. Thoughts @fk @shannonbux @ryanditjia? |
I agree with my implementation being too prominent! I will tone it down by shortening to concise sentences that you suggested. And, even better, we can replace blockquote with the Butler component. I’ve suggested implementing Butler + MDX in the issue, let’s wait for the responses! |
@ryanditjia it'd be great to get a first version of this merged, then maybe it can be re-visited once there's a proper Butler callout component. If you have time to finish this up then I think the least work to getting it merged would be:
I think that's it! |
Apologies for letting this go stale for so long. Seems like it’s easier for me to close this and send another PR, doing that now! |
Refs: