Gatsby: boundActionCreators warning should say to replace them with "actions" and also link to the upgrade docs for this

[marisamorby]

boundActionCreators warning should say to replace them with "actions" and also link to the upgrade docs for this

[marisamorby]

@KyleAMathews: I updated this from a card to an issue so that it could be moved into the appropriate sub-epic.

[jlengstorf]

We should link to https://next.gatsbyjs.org/docs/migrating-from-v1-to-v2/#rename-boundactioncreators-to-actions in the error message.

The error message is generated here:

https://github.com/gatsbyjs/gatsby/blob/b6bdb82555893177a6c85d62b28abe471e484b8d/packages/gatsby/src/commands/develop.js#L388-L421

I think we can modify it a bit:

    const fixMap = {
      boundActionCreators: {
        newName: `actions`,
        docsLink: `https://next.gatsbyjs.org/docs/migrating-from-v1-to-v2/#rename-boundactioncreators-to-actions`,
      },
      // ...
    }

And then update the print template to show that docs link as well.

This would be a great first issue for someone!

[thefifthisa]

Hi, I'm new here and I'd love to take this!

[jlengstorf]

Yes! Thanks @thefifthisa! Let us know if you have questions or need a second set of eyes. 💪💜

[thefifthisa]

@jlengstorf Just opened a PR! Please let me know what I need to change or if it's okay already.