Skip to content
This repository has been archived by the owner on Jul 15, 2020. It is now read-only.

Question: Add next-seo to starter? #90

Open
garmeeh opened this issue Aug 28, 2018 · 2 comments
Open

Question: Add next-seo to starter? #90

garmeeh opened this issue Aug 28, 2018 · 2 comments

Comments

@garmeeh
Copy link

garmeeh commented Aug 28, 2018

Hi @iaincollins, I created a little plugin for helping with SEO and was wondering would you see any benefit of adding it as part of the Next.js Starter project? If so I would be happy to start working on a PR to add it.

You can find the plugin here.

@iaincollins
Copy link
Owner

Hey Gary, oh this is neat! I should probably be removing some cruft from the project, but this is a great module to know about and I will be using for sure; I work on number of projects and always end up stuffing the markup in my layout and it didn't occur to me to split it off.

The JSON LD stuff looks interesting too!

I might not have time right now, but if you would like to do a pull request to add it I'll accept, otherwise I'll get round to it when I can. :-)

Thanks for the issue and the module!

@garmeeh
Copy link
Author

garmeeh commented Aug 29, 2018

😄 nice one. I will work on a PR and get your feedback on how it should be incorporated from there.

@garmeeh garmeeh mentioned this issue Aug 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants