fix(plugin-netlify-cms): use 'netlify-identity.js' instead of 'netlify-identity-widget.js'

[erezrokah]

Fixes #22386

Netlify Identity Widget provides two build outputs:

1. netlify-identity-widget.js which initializes the widget on DOMContentLoaded.
2. netlify-identity.js which doesn't initialize the widget.

This PR switches from number 1 to number 2 to avoid duplicate initialization as the widget is already initialized here

gatsby/packages/gatsby-plugin-netlify-cms/src/cms-identity.js

Line 24 in 9e291e9

netlifyIdentityWidget.init()

[pieh]

There are some other references to netlify-identity-widget ( https://github.com/gatsbyjs/gatsby/blob/561d33e2e491d3971cb2a404eec9705a5a493602/packages/gatsby-plugin-netlify-cms/src/gatsby-browser.js ). Do those need to be adjusted as well?

[erezrokah]

There are some other references to netlify-identity-widget ( https://github.com/gatsbyjs/gatsby/blob/561d33e2e491d3971cb2a404eec9705a5a493602/packages/gatsby-plugin-netlify-cms/src/gatsby-browser.js ). Do those need to be adjusted as well?

Good question, the import statements (and any other references to the package name) should remain the same as the package imported is named netlify-identity-widget.
The change tells WebPack to use a different build output from the same package.
Hopefully that makes things more clear.

[pieh]

Good question, the import statements (and any other references to the package name) should remain the same as the package imported is named netlify-identity-widget.
The change tells WebPack to use a different build output from the same package.
Hopefully that makes things more clear.

Oh, derp. The additional abstraction there threw for a loop ;) Yeah definitely see how it works now

[pieh]

This makes sense, thanks!