mobx-react-router
mobx-react-router copied to clipboard
alisd23
build: upgrade
- rewrite source by ts.
- compatible with mobx v6.
- use rollup for module and cmd output.
- add module output.
- change declaration file path to module/index.d.ts.
- change type,module,browser,unpkg section in package.json.
- upgrade eslint and babel dependencies.
- update test case.
may be it mobx-react-router should upgrade to v5.
It looks like this library isn't maintained anymore. Would you like to create a separate NPM package?
@kubk already published https://www.npmjs.com/package/@superwf/mobx-react-router
I have tried to use this patch with Mobx6, but the UMD build in IE11 gives syntax errors. You can find the similar error in
https://github.com/react-dnd/react-dnd/issues/1557
The bundle should be transpiled to IE11 compatible using babel pre-set
See the change in rollup.config.js
https://github.com/react-dnd/react-dnd/pull/1621/commits/242a6e177405ed046ce77575921bbc001825156a
@jimbofreedman @eduardofcgo @alisd23 can you guys check this / merge so we have MobX v6 support?
@eek You can use the aforementioned fork for now. The original author of this library responds neither here nor to emails.
@kubk I am using the other repo at the moment, but in the other issue - https://github.com/alisd23/mobx-react-router/issues/74 @alisd23 gave full access to @jimbofreedman and @eduardofcgo to keep the repo updated. Maybe he should have also given permissions to @superwf
@eek Great, let's wait for them. I've been using this fork for the past 2 months on production and it works with no issues.
@superwf you did such great work with typescript and rollback. Are you still interested in merging some of your changes?
