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

Add optional custom frontmatter meta description for SEO and Open Graph #146

Closed
wants to merge 1 commit into from
Closed

Add optional custom frontmatter meta description for SEO and Open Graph #146

wants to merge 1 commit into from

Conversation

nielsenjared
Copy link

@nielsenjared nielsenjared commented Dec 14, 2018

I appreciate the new SEO merge in #143. An additional feature I thought useful would be a custom meta description frontmatter field for SEO and Open Graph purposes. I'm relatively new to Gatsby, so I hope this PR is useful and not interfering with anything under the hood. Cheers!

Changes proposed:

  • To src/pages/index.js and src/templates/blog-post.js add description to frontmatter query
  • Add short-circuit evaluation for frontmatter.description in posts mapping (index.js) and SEO component (blog-post.js)
  • Add description field to hi-folks/index.md to demonstrate use

@nielsenjared nielsenjared changed the title Add option for custom frontmatter meta descriptions for SEO and Open Graph Add optional custom frontmatter meta description for SEO and Open Graph Dec 17, 2018
@sidharthachatterjee
Copy link
Contributor

Hey @nielsenjared

Thank you for opening this. We've moved our official starters over to our monorepo to help ease maintenance (gatsbyjs/gatsby#10310). Closing this since this repo is now a read-only mirror.

Please reopen this PR against the monorepo if you'd like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants