gatsby-theme-material-ui icon indicating copy to clipboard operation
gatsby-theme-material-ui copied to clipboard

Usage with gatsby-plugin-next-seo?

Open jakubjafra opened this issue 4 years ago • 0 comments

Hi! :)

There is a neat new package called gatsby-plugin-next-seo with uses react-helmet-async instead of react-helmet. This plugin, however, requires you to use react-helmet. Is there a way to lift that requirement? I'd like to use any helmet lib.

Right now, when using both react-helmet's there are errors, like - for example - <title> is rendered empty.

I tried switching to react-helmet-async by overwriting gatsby-theme-material-ui-top-layout/components/viewport.js and replacing react-helmet with react-helmet-async but in the gatsby-node.js the gatsby-plugin-react-helmet is still required. I ultimately resolved this issue by using patch-package and removing those entries locally.

I could post a pull request with a change, but I'm unsure what's the best way to proceed.

Thanks

jakubjafra avatar Jan 25 '21 19:01 jakubjafra