Skip to content

Conversation

@amberleyromo
Copy link
Contributor

Tweaks reviewing overview section for v2. Consistency in the page title/sidebar.

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 good!

## Hybrid app pages

With this method, Gatsby renders the initial page with shared page content -- then when your React components load in the browser, they can fetch and render data from APIs. The [React docs have a simple example of how to do this.](https://reactjs.org/docs/faq-ajax.html)
With this method, Gatsby initially renders the page statically, and then when your React components load in the browser they can fetch and render data from APIs.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is kinda awkward language — perhaps change it to describe how a visitor to a Gatsby site first loads an HTML file and then the JavaScript.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified the language

@KyleAMathews
Copy link
Contributor

KyleAMathews commented Aug 7, 2018

Deploy preview for using-postcss-sass failed.

Built with commit 2f5e1ca

https://app.netlify.com/sites/using-postcss-sass/deploys/5b6a297e73f2cf0343501caf

@gatsbybot
Copy link
Collaborator

gatsbybot commented Aug 7, 2018

Deploy preview for using-drupal ready!

Built with commit 2f5e1ca

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

@gatsbybot
Copy link
Collaborator

gatsbybot commented Aug 7, 2018

Deploy preview for gatsbygram ready!

Built with commit 2f5e1ca

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

@KyleAMathews KyleAMathews merged commit 1b0aeab into gatsbyjs:master Aug 8, 2018
@KyleAMathews
Copy link
Contributor

Thanks!

m-allanson added a commit that referenced this pull request Aug 8, 2018
* master: (21 commits)
  Disable www-graphql-docker-push.sh (#7127)
  chore(release): Publish
  Fix onRouteUpdate & onPreRouteUpdate (#7142)
  chore(release): Publish
  Fix showing dev 404 page when there isn't one in development (#7140)
  Replace 'serve' package w/ express-static to restore compression (#7139)
  Fix tests (#7138)
  Add delay to cyress test where cyress kept failing
  chore(release): Publish
  (#6581): v2 docs building apps with gatsby overview update (#7115)
  Added my blog to sites (#7125)
  Add babel-plugin-macros (#7129)
  Update babel.md (#7012)
  (#6590): flesh out preparing your environment overview stub (#7137)
  [gatsby-plugin-netlify] update static cache header to match docs (#7132)
  (#7118): Clarify prettier instructions and update tutorial sidebar (#7130)
  Fixing spelling error (#7136)
  chore(release): Publish
  Try publishing again
  chore(release): Publish
  ...

# Conflicts:
#	yarn.lock
m-allanson added a commit that referenced this pull request Aug 8, 2018
* master: (566 commits)
  chore(release): Publish
  Fix mistakenly identifying files as using commonjs when they weren't (#7153)
  Update collectPackages to getPackages for new lerna rc (#7126)
  Update index.md (#7145)
  Disable www-graphql-docker-push.sh (#7127)
  chore(release): Publish
  Fix onRouteUpdate & onPreRouteUpdate (#7142)
  chore(release): Publish
  Fix showing dev 404 page when there isn't one in development (#7140)
  Replace 'serve' package w/ express-static to restore compression (#7139)
  Fix tests (#7138)
  Add delay to cyress test where cyress kept failing
  chore(release): Publish
  (#6581): v2 docs building apps with gatsby overview update (#7115)
  Added my blog to sites (#7125)
  Add babel-plugin-macros (#7129)
  Update babel.md (#7012)
  (#6590): flesh out preparing your environment overview stub (#7137)
  [gatsby-plugin-netlify] update static cache header to match docs (#7132)
  (#7118): Clarify prettier instructions and update tutorial sidebar (#7130)
  ...

# Conflicts:
#	yarn.lock
porfirioribeiro pushed a commit to porfirioribeiro/gatsby that referenced this pull request Aug 22, 2018
…atsbyjs#7115)

* (gatsbyjs#6581): v2 docs building apps with gatsby overview update

* (gatsbyjs#6581): tweak hybrid app language
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.

3 participants