New syntax needed with "history"

Question

New syntax needed with "history"

AlexandraBeautyman opened this issue 5 years ago · comments

Alexandra Beautyman commented 5 years ago

I get these two browser errors:

Warning: Please use require("history").createBrowserHistory instead of require("history/createBrowserHistory"). Support for the latter will be removed in the next major release.

warnAboutDeprecatedCJSRequire.js:17 Warning: Please use require("history").createMemoryHistory instead of require("history/createMemoryHistory"). Support for the latter will be removed in the next major release.

Finn Terdal · Answer 1 · Wed May 22 2019 02:23:39 GMT+0800 (China Standard Time)

Yep, users will see this deprecation warning, and it's quite confusing, cause the boilermaker code doesn't use CommonJS-style require, it uses ES6 import statements.

I'll submit a quick PR to fix this. Thanks for making an issue!