-
Notifications
You must be signed in to change notification settings - Fork 2.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
Nits on the Getting Started page #5778
Comments
I can take this one. |
Thanks that'd be great @reginareynolds, looking forward to your PR! |
This is going to sound like a silly question, but where did the website folder go? According to the Contributing document, it's supposed to be in the root directory, but it's not. Did it get moved to the _website folder in the js directory? |
Hi @reginareynolds oh wow good catch! Yes, the website is now located at Thanks so much! |
Hi @abidlabs is this issue still open. |
Yea @Luwa-Tech ! If you'd like to open a PR, please feel free |
@freddyaboulton I want to work on the issue, I will be opening a PR by today |
There are too many separate concepts in the "Sharing Your App" section of the getting started guide.
My Recommendation:
Mounting within another...
should be under theSharing Your App
HeaderAuthentication
into another section and includePassword Protected...
andOAuth
into that sectionAccessing the Network Request
feels pretty random here. Maybe move toKey Features
?Security
should pulled out into its own section.Api Usage
should be pulled out into its own section.The text was updated successfully, but these errors were encountered: