Replace gatsby-plugin-react-helmet with native Gatsby Head

[nschonni]

Enter your suggestions in details:

During building, there is a warning now, because the functionaity is now built into Gatsby

warning gatsby-plugin-react-helmet: Gatsby now has built-in support for modifying the document head. Learn more at https://gatsby.dev/gatsby-head

Seems like the SEO section that uses it currently also has some updated guidance on the Gatsby docs

[ovflowd]

@nschonni we actually use the built-in feature from Gatsby. As you can see we don't even import the react-helmet plugin.

The issue here is that one of our plugins has the react-helmet as a dependency on their "package.json", I think that's why Gatsby's build is complaining.

But we're actually not using it.

[nschonni]

https://github.com/nodejs/nodejs.dev/blob/2cfa1b721d4550a9215caa31148161e1518075ca/src/components/Seo/index.tsx#L2

[ovflowd]

@nschonni there's nothing wrong with that line. Again, what I was telling you, is that we don't use the gatsby-react-helmet plugin, which is correct.

We still, yes need to move to what they introduced on 4.19.0 (which is quite recent).

[mstomar698]

Is this issue open or not? And please do tell what is to be done?

[ovflowd]

The issue is open. It's literally on its title. What needs to be done is basically remove the usage of react-helmet.

But it needs testing if it will work out of the box

[mstomar698]

ok

[ovflowd]

@nschonni I made some research, and accomplishing this will be rather harder than easy. Removing Hacktoberfest label as this is not a straightforward issue.