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:

gatsby/packages/gatsby/src/commands/develop.js

Lines 388 to 421 in b6bdb82

	function printDeprecationWarnings() {
	const deprecatedApis = [`boundActionCreators`, `pathContext`]
	const fixMap = {
	boundActionCreators: `actions`,
	pathContext: `pageContext`,
	}
	const deprecatedLocations = {}
	deprecatedApis.forEach(api => (deprecatedLocations[api] = []))
	glob
	.sync(`{,!(node_modules|public)/**/}*.js`, { nodir: true })
	.forEach(file => {
	const fileText = fs.readFileSync(file)
	const matchingApis = deprecatedApis.filter(
	api => fileText.indexOf(api) !== -1
	)
	matchingApis.forEach(api => deprecatedLocations[api].push(file))
	})
	deprecatedApis.forEach(api => {
	if (deprecatedLocations[api].length) {
	console.log(
	`%s %s %s %s`,
	chalk.cyan(api),
	chalk.yellow(`is deprecated. Use`),
	chalk.cyan(fixMap[api]),
	chalk.yellow(`instead. Check the following files:`)
	)
	console.log()
	deprecatedLocations[api].forEach(file => console.log(file))
	console.log()
	}
	})
	}

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.