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

chore(docs): Adapters #38233

Merged
merged 16 commits into from
Aug 24, 2023
Merged

chore(docs): Adapters #38233

merged 16 commits into from
Aug 24, 2023

Conversation

LekoArts
Copy link
Contributor

@LekoArts LekoArts commented Jun 14, 2023

Description

RFC: #38231

Docs changes for feature PR: #38232

Sidebar PR: https://github.com/gatsby-inc/dotcom/pull/846

Related Issues

#38232

@LekoArts LekoArts added the type: documentation An issue or pull request for improving or updating Gatsby's documentation label Jun 14, 2023
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jun 14, 2023
@LekoArts LekoArts added this to the Adapters milestone Jun 14, 2023
@LekoArts LekoArts removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jun 14, 2023
Copy link
Contributor

@codebyuma codebyuma left a 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/adapters.md Outdated Show resolved Hide resolved
docs/docs/how-to/previews-deploys-hosting/adapters.md Outdated Show resolved Hide resolved
docs/docs/how-to/previews-deploys-hosting/adapters.md Outdated Show resolved Hide resolved
docs/docs/how-to/previews-deploys-hosting/adapters.md Outdated Show resolved Hide resolved
docs/docs/how-to/previews-deploys-hosting/caching.md Outdated Show resolved Hide resolved
docs/docs/reference/config-files/gatsby-config.md Outdated Show resolved Hide resolved
docs/docs/reference/config-files/gatsby-config.md Outdated Show resolved Hide resolved
LekoArts and others added 5 commits June 28, 2023 09:10
@pieh pieh marked this pull request as ready for review August 23, 2023 14:22
@pieh pieh merged commit 7c6a7af into master Aug 24, 2023
6 checks passed
@pieh pieh deleted the docs/gatsby-adapters branch August 24, 2023 09:53
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.

4 participants