-
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
feat(blog): add localization with gatsby and sanity #11684
feat(blog): add localization with gatsby and sanity #11684
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.
This is looking great. Generally I'm not quite sure whether this fits on the blog (and how we categorize what is blog-post worthy) because it's possibly a bit more focused than other content we've previously displayed on our blog.
That said--it's a great blog post, so I think we'd like to have this content somewhere (or link to it?), even if not our blog.
@marcysutton any ideas or suggestions here?
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
Thanks I appreciate the feedback for sure. To be honest the only reason I wrote it was because I got a request to do so on twitter from @jlengstorf. ;) |
Follow up after my edits... no idea why its failing the CI now. :( |
…anity-with-localization
@williamtstanley don't think it's actually failing! If multiple commits come in, it auto-cancels the older/redundant builds! |
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.
@DSchau @marcysutton I would say this is great blog post material.
This is great content, and there's only two options here, blog post or docs.
It would be tough to make this a canonical docs article, because we only have standalone canonical articles on one topic (eg Gatsby + Sanity or Gatsby + Localization) rather than the intersection of two topics (Gatsby + Sanity + Localization). If it was a blog post, however, we can link to it from our l10n / i18n articles.
Therefore, this should be a blog post.
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.
This is great, thanks so much for putting it together! I agree that it makes sense as a blog post...my only question is how it differs from the other Sanity post. What new information does this post provide, and can that go into the title somehow?
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
docs/blog/2019-03-01-localization-with-gatsby-and-sanity/index.md
Outdated
Show resolved
Hide resolved
Combine two sentence fragments to form a complete sentence. Fix a typo and expend the link text to be more clear what the link was pointing to. Also updated the working title to "A method of localization with gatsby and sanity.io" <-- please suggest other options I am not in love with this.
…anity-with-localization
@marcysutton I fixed up the merge conflict--we can merge posts ahead of their scheduled post date (currently March 1st) and they won't go live until that date. Anything else that needs fixing up? Thanks for the patience here @williamtstanley! |
Love it! |
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.
Let's get this merged. It'll go live March 1st--this Friday!
(Holy cow, it's almost March!)
Holy buckets, @williamtstanley — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
Unfortunately I found an issue with my post when considered against this bug. #5255 |
Description
Blog post: A method of doing localization with Gatsby and Sanity.io