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): Update tutorial part 1 to reflect changes to add site flow #34427

Merged

Conversation

aghreed
Copy link
Contributor

@aghreed aghreed commented Jan 7, 2022

Description

Now that we have enabled the new add site experience for all users, it follows that we should update the instructions in the tutorial that reference that flow.

Related Issues

sc-42765

Testing

Many thanks to @rmatambo8 for discovering the indentation issue here that was plaguing me! All looks good to me now locally!

Screen Shot 2022-01-11 at 4 16 58 PM

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jan 7, 2022
@aghreed aghreed added type: documentation An issue or pull request for improving or updating Gatsby's documentation and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jan 7, 2022
@aghreed aghreed marked this pull request as draft January 7, 2022 16:11
@aghreed aghreed marked this pull request as ready for review January 10, 2022 22:49
rmatambo8
rmatambo8 previously approved these changes Jan 11, 2022
Copy link
Contributor

@rmatambo8 rmatambo8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's amazing how many files needed changing for this.

LGTM!


<Announcement style={{marginBottom: "1.5rem"}}>

**Note:** If you are part of more than one GitHub organization, you will need to first select the organization with which the repository resides at this step before selecting the repository itself.
**Note:** If you are part of more than one GitHub organization, you will need to first select the organization with which the repository resides at this step before selecting the repository itself.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, this indentation looks suspect to me – could also be related to the Announcement component being using inside the parent ol

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the latest commit on this branch by Rodney, I'm seeing everything work as expected. He is my personal hero today ❤️

Copy link
Contributor

@rmatambo8 rmatambo8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@aghreed aghreed merged commit 08ce379 into master Jan 11, 2022
@aghreed aghreed deleted the aghreed/sc-42765/update-tutorial-references-to-add-site-flow branch January 11, 2022 21:42
moonmeister added a commit to moonmeister/gatsby that referenced this pull request Jan 12, 2022
* master:
  fix(gatsby-core-utils): Re-Export updateSiteMetadata (gatsbyjs#34462)
  fix(deps): update starters and examples gatsby packages to ^4.5.1 (gatsbyjs#34463)
  fix(gatsby-dev-cli): Do not ignore engines (gatsbyjs#34461)
  fix(gatsby): Revert gatsbyjs#33786 (gatsbyjs#34458)
  fix(deps): update starters and examples - gatsby (gatsbyjs#34449)
  chore(changelogs): update changelogs (gatsbyjs#34451)
  Update tutorial part 1 to reflect changes to add site flow (gatsbyjs#34427)
  preview 2.0 UI changes (gatsbyjs#34440)
@rmatambo8 rmatambo8 changed the title Update tutorial part 1 to reflect changes to add site flow chore(docs): Update tutorial part 1 to reflect changes to add site flow Jan 13, 2022
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.

3 participants