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

Add "sourcing from Contentful" section. #8861

Merged

Conversation

alejandronanez
Copy link
Contributor

@alejandronanez alejandronanez commented Oct 6, 2018

WIP to close #8741

So far I've done:
- Copied over https://www.gatsbyjs.org/blog/2018-1-25-building-a-site-with-react-and-contentful/#what-is-contentful-why-choose-it I still need to remove/add what's needed

Will continue with this work over the next few days!

@shannonbux 👋 what should be the right labels for this besides type: documentation? I'm still learning the ropes here 🙂

Update

Reasy for review.

Closes #8714

@alejandronanez alejandronanez added the type: documentation An issue or pull request for improving or updating Gatsby's documentation label Oct 6, 2018
@alejandronanez alejandronanez requested a review from a team October 6, 2018 02:59
@amberleyromo amberleyromo self-assigned this Oct 8, 2018
@amberleyromo
Copy link
Contributor

@alejandronanez Don't worry about the labels, you're good!

Ping when you're ready for review. Quick note -- it looks like you used the full content of the blog post. It'd probably be a good start to axe the part before the What is Contentful? Why choose it? section of the post.

Thanks for working on this!

@amberleyromo
Copy link
Contributor

Hi @alejandronanez -- this still has some "todos" listed at the bottom. Still WIP?

@alejandronanez
Copy link
Contributor Author

Hi @alejandronanez -- this still has some "todos" listed at the bottom. Still WIP?

@amberleyromo do you think I should add a contentful real-world example to this doc? or is it out of the scope of this PR?

If it is out of scope, this would just need a spell check and that's it!

@amberleyromo
Copy link
Contributor

@alejandronanez I think for the sake of this PR it's out of scope -- let's make any final edits you want and get this merged!

If you'd like to get this merged and then open an additional PR to flesh it out further, that would be much appreciated!

@alejandronanez alejandronanez changed the title WIP: Add "sourcing from Contentful" section. Add "sourcing from Contentful" section. Oct 20, 2018
@alejandronanez
Copy link
Contributor Author

Hello @amberleyromo! This is ready for review now :)

@amberleyromo amberleyromo merged commit 8e27d72 into gatsbyjs:master Oct 22, 2018
@shannonbux
Copy link
Contributor

This looks awesome @alejandronanez! Way to follow the headers specified in the Style Guide 👍

As far as real world examples go, you could add a section at the bottom titled "Other resources" or "further reading" to link to a blogpost, source code of an example site, outside tutorials, etc.

@alejandronanez
Copy link
Contributor Author

@shannonbux thanks for the tips! 👍🏼👍🏼👍🏼

@shannonbux
Copy link
Contributor

@alejandronanez Also just realized I didn't make this clear--I know this PR already got merged, and so adding a new section at the bottom would have to happen in a new PR :) Again, way to go!

@alejandronanez alejandronanez deleted the add-contentful-documentation branch October 24, 2018 01:26
gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
* Issue 8741: WIP
Copied over https://www.gatsbyjs.org/blog/2018-1-25-building-a-site-with-react-and-contentful/#what-is-contentful-why-choose-it I still need to remove/add what's needed

* Removing unuseful information from the original post!

* docs: clean up documentation and mention the usage of gatsby-source-contentful plugin

* Removed TODO list

* Tweaks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation An issue or pull request for improving or updating Gatsby's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Content migration - fill out "Sourcing from Contentful" stub article [feat]: add gatsby-source-sql
3 participants