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.

Sign up for GitHub

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

Jump to bottom

chore(docs): Old occurrences of gatbyjs.org #34402

Merged
merged 3 commits into from
Jan 7, 2022
Merged

chore(docs): Old occurrences of gatbyjs.org #34402

merged 3 commits into from
Jan 7, 2022

Conversation

AnilSeervi
Copy link
Contributor

@AnilSeervi AnilSeervi commented Jan 5, 2022 •
edited
Loading

Description

  • Fix instances of gatsby.org
  • Typo: gatbyjs.org -> gatsbyjs.com
  • fix 404 links

Documentation

Related Issues

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jan 5, 2022
@tyhopp
Copy link
Contributor

tyhopp commented Jan 5, 2022

Hi @AnilSeervi, thanks for the PR!

It looks like the CI didn't run on your PR, could you push an empty commit or enable us to make edits so that we can have it run?

I also had a look at the docs subdir and noticed there were other occurrences of gatsbyjs.org (such as this occurrence) that we can change (ignoring store and example urls like using-gatsby-image). If you're interested, it would be great if you could help also change those.

@tyhopp tyhopp added status: awaiting author response Additional information has been requested from the author 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 5, 2022
@LekoArts LekoArts changed the title ✏️ Typo: gatbyjs.org -> gatsbyjs.com chore(docs): Old occurrences of gatbyjs.org Jan 5, 2022
@AnilSeervi
Copy link
Contributor Author

Hi @AnilSeervi, thanks for the PR!

It looks like the CI didn't run on your PR, could you push an empty commit or enable us to make edits so that we can have it run?

I also had a look at the docs subdir and noticed there were other occurrences of gatsbyjs.org (such as this occurrence) that we can change (ignoring store and example urls like using-gatsby-image). If you're interested, it would be great if you could help also change those.

I went ahead and fixed all occurences of gatbsy.org in /docs

Also fixed some 404 links while I was at it ✨

tyhopp
tyhopp approved these changes Jan 6, 2022
View reviewed changes
Copy link
Contributor

@tyhopp tyhopp left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @AnilSeervi!

@LekoArts LekoArts removed the status: awaiting author response Additional information has been requested from the author label Jan 6, 2022
tyhopp
tyhopp suggested changes Jan 6, 2022
View reviewed changes
Copy link
Contributor

@tyhopp tyhopp left a comment

Choose a reason for hiding this comment

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

I did find one small thing, could you please use relative links for the case above? Thanks!

@AnilSeervi AnilSeervi requested a review from tyhopp January 6, 2022 10:34
@tyhopp tyhopp merged commit dda64c3 into gatsbyjs:master Jan 7, 2022
moonmeister added a commit to moonmeister/gatsby that referenced this pull request Jan 11, 2022
@moonmeister
Merge branch 'master' into fix/validate/warnonunknown
eb2a22a 
* master: (24 commits)
  chore(docs): Release Notes v4.5 (gatsbyjs#34425)
  chore(docs): Update quick-start guide (gatsbyjs#34445)
  chore(docs) : Typo fix GatbsyImage -> GatsbyImage (gatsbyjs#34439)
  perf(gatsby): reuse rootNode & trackedRootNodes caches across instances of graphqlRunner (gatsbyjs#33695)
  Update media-item-processing.md (gatsbyjs#34434)
  chore(docs): Update localization doc (gatsbyjs#34429)
  test(ssr): Fix flakes (gatsbyjs#34443)
  chore(release): Publish next
  Revert "docs: Match egghead.io video instructions (gatsbyjs#34315)" (gatsbyjs#34384)
  fix(gatsby-plugin-manifest): generate icons sequentially (gatsbyjs#34331)
  Fix misspelling of "precedence" in log message (gatsbyjs#34428)
  chore(docs): Adjust doc mentions of gatsby-plugin-create-client-paths (gatsbyjs#34424)
  chore(docs): Update static-folder doc (gatsbyjs#34392)
  Upgrade to strip-ansi ^6.0.1 (gatsbyjs#34383)
  chore(gatsby-plugin-create-client-paths): Update client paths plugin readme with migration info (gatsbyjs#34423)
  chore: Remove deprecated client paths plugin references (gatsbyjs#34422)
  chore(docs): Old occurrences of gatbyjs.org (gatsbyjs#34402)
  Update plugins.md to have correct URL for gatsby-plugin-segment-js (gatsbyjs#34397)
  chore(gatsby): Give option to ignore output from workers and silence validate-engines (gatsbyjs#34416)
  chore(release): Publish next pre-minor
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyhopp tyhopp tyhopp approved these changes

Assignees
No one assigned
Labels
type: documentation An issue or pull request for improving or updating Gatsby's documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AnilSeervi @tyhopp @LekoArts