-
Notifications
You must be signed in to change notification settings - Fork 10.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
chore(docs): Adapters #38233
chore(docs): Adapters #38233
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @LekoArts, I just finished my first review. There is a lot of great content here! For the most part, it's easy to follow and I thought the examples were really helpful.
I left a number of suggestions for your consideration - mostly around grammar or to make something a bit clearer. I also left a few questions for you and one or two suggestions for adding links or information in other places.
Hopefully the suggestions are clear but I'm happy to jump on a call to discuss or pair if needed, just let me know.
docs/docs/how-to/previews-deploys-hosting/zero-configuration-deployments.md
Outdated
Show resolved
Hide resolved
docs/docs/how-to/previews-deploys-hosting/creating-an-adapter.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Uma Chandran <codebyuma@users.noreply.github.com>
Co-authored-by: Uma Chandran <codebyuma@users.noreply.github.com>
Description
RFC: #38231
Docs changes for feature PR: #38232
Sidebar PR: https://github.com/gatsby-inc/dotcom/pull/846
Related Issues
#38232