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

[docs][workflows] building apps with Gatsby #18993

Closed
5 tasks done
gillkyle opened this issue Oct 24, 2019 · 7 comments
Closed
5 tasks done

[docs][workflows] building apps with Gatsby #18993

gillkyle opened this issue Oct 24, 2019 · 7 comments
Assignees
Labels
type: documentation An issue or pull request for improving or updating Gatsby's documentation

Comments

@gillkyle
Copy link
Contributor

gillkyle commented Oct 24, 2019

Part of the Top 25 Learning Workflows initiative. See #13708 for the meta issue that this issue falls under.

User story

As a new Gatsby user, I want to understand how to build apps using Gatsby.

Evaluation

Search Discover Complete Linked Tone Style Overall
😄 😐 😞 😐 😄 😄 😐

Steps taken to implement

  1. Searchability:

Gatsby docs are surfaced first in search results and seem to be identifiable by common search terms associated with apps (even though the exact terminology can vary).

  1. Discoverability:

    • Building Apps with Gatsby doc isn't in the sidebar (like mentioned above), but finding docs like "Adding Authentication" only takes 3 clicks from the docs
    • docs on building apps, auth, and dynamic content can be found okay from the .org site search
  2. Completeness:

    • client only routes and authentication pages continue to have complaints about not being easy to understand, there could be much more illustrative examples for it with real use cases
    • authentication guide links off to the client only route page as well as the authentication tutorial (but with an erroneous subheader in the link) as it's only 2 bullet points for "Setting the authentication workflow", more content here would be beneficial
    • client data fetching offers good examples and demonstrates how to perform the task well 👍 however, I think the value of this guide might more accurately be stated as "Fetching Data" since it explains both build time and client side data fetching, which might help more people land on this guide

A general trend in these guides seems to be linking off to example repos and places where these patterns are used rather than actually teaching them inline with the docs and code examples

  1. Linkedness

    • Seems well interlinked 👍
  2. Tone

    • The tone feels positive, it's often really terse, explanations could be a bit wordier
  3. Style

    • Conforms with the style guide 👍

Recommendations

I'll likely break these recommendations into a few contextual sections (and probably in this order):

@gillkyle gillkyle added the type: documentation An issue or pull request for improving or updating Gatsby's documentation label Oct 24, 2019
@gillkyle gillkyle self-assigned this Oct 24, 2019
@ghost
Copy link

ghost commented Oct 25, 2019

I can "add the Building Apps with Gatsby page to the sidebar" by just doing a PR as well as a few of the others as well if want🤷‍♀️☺️

@ghost ghost self-assigned this Oct 25, 2019
@gillkyle
Copy link
Contributor Author

Thanks for offering the help @bvlktech! I think with these recommendations it'd make sense to keep most of them in one PR so that the changes can be seen side by side in the context with where docs are moving and how pages are being renamed. Since it could impact the general information architecture and I'm guessing some of the pages will be able to be combined/moved.

I'll hopefully be able to get a PR up soon that maybe you'd like to look over and review?

@ghost
Copy link

ghost commented Oct 25, 2019

Sure thing cant wait to check it out☺️

@ghost
Copy link

ghost commented Oct 31, 2019

did u make the PR yet? jw is all

@gillkyle
Copy link
Contributor Author

gillkyle commented Nov 1, 2019

@bvlktech I just made a first PR at #19214. I've been doing a lot of ideation around this the last few days because it's a confusing topic for a lot of people. That first PR should cover some of the groundwork around the overview and conceptualizing making app features possible -- with a doc on hydration as well.

@ghost
Copy link

ghost commented Nov 4, 2019

yes i saw will check it out for further review thanks for getting back!

@marcysutton
Copy link
Contributor

We can close this issue now, as the work has mostly been completed! The outstanding issue in #20050 can be tackled separately. Great work @gillkyle!

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

No branches or pull requests

2 participants