search

iaincollins / nextjs-starter

A starter project for Next.js with authentication
https://nextjs-starter.now.sh
ISC License
1.37k stars 422 forks source link

Question: Add next-seo to starter? #90

Open garmeeh opened 6 years ago

garmeeh commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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