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

Standardizing headers, tweaked wording #4375

Merged
merged 2 commits into from
Mar 16, 2018

Conversation

shannonbux
Copy link
Contributor

While re-reading this, I'm thinking the order of sections could be a bit hard to digest. (components, global styles, typography.js, plugins. @amberleyromo what order is sensible to you? Usually the most basic pathway is the best, so is there anything else we ought to cut out or re-order to make it more basic?

While re-reading this, I'm thinking the order of sections could be a bit hard to digest. (components, global styles, typography.js, plugins. @amberleyromo what order is sensible to you? Usually the most basic pathway is the best, so is there anything else we ought to cut out or re-order to make it more basic?
@ghost ghost assigned shannonbux Mar 5, 2018
@ghost ghost added the review label Mar 5, 2018
@KyleAMathews KyleAMathews self-requested a review March 5, 2018 20:44
Copy link
Contributor

@KyleAMathews KyleAMathews left a comment

Choose a reason for hiding this comment

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

Looks great!

@gatsbybot
Copy link
Collaborator

gatsbybot commented Mar 5, 2018

Deploy preview for gatsbygram ready!

Built with commit 97c2757

https://deploy-preview-4375--gatsbygram.netlify.com

@amberleyromo
Copy link
Contributor

@shannonbux re: the sensible order, some tweaks along the following lines are what make sense to me. This is mostly a very slight reordering, and a very slight change of focus in the way it's explained.

  1. Development environment (by getting in place all of the necessary pieces, which would include succinct explanations of concepts (what is a terminal, and why do you need to interface with it, etc).
  2. Introduction to gatsby: goal to scaffold out the starter from the gatsby cli, and have a high-level tour of what ships with it -- what are all these pieces?
  3. Thinking in components: here introducing the concepts of components and layouts, again by pointing to the concrete example of the starter scaffold.
  4. Styling: Here we can introduce some CSS concepts, probably starting with some plain css, and then making the jump to libraries like Typography.js, etc.
  5. Data flow: Instead of making it about learning GraphQL, start with -- the problem we're solving is how to get data from point A to point B. A high-level introduction to the Gatsby style data flow, then some nitty gritty with GraphQL. GraphQL is the tool -- focus on the why, first.

@shannonbux
Copy link
Contributor Author

shannonbux commented Mar 7, 2018 via email

@amberleyromo
Copy link
Contributor

Exactly. The problem we're solving is handling data. The tool is GraphQL :)

@shannonbux
Copy link
Contributor Author

shannonbux commented Mar 8, 2018 via email

@m-allanson
Copy link
Contributor

Hey @shannonbux does this still need to go in, or was the work covered in your other recent PRs?

@amberleyromo
Copy link
Contributor

@shannonbux these changes don't touch what we discussed today (some minimal in the intro). I think go ahead and merge, and we'll just do fresh changes from there.

@shannonbux shannonbux merged commit 145f062 into master Mar 16, 2018
@ghost ghost removed the review label Mar 16, 2018
@shannonbux
Copy link
Contributor Author

Thanks everybody!

@shannonbux shannonbux deleted the tutorial-part-two-enhancements branch March 16, 2018 23:38
@gatsbybot
Copy link
Collaborator

Deploy preview for using-drupal ready!

Built with commit 97c2757

https://deploy-preview-4375--using-drupal.netlify.com

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.

5 participants