Please release this fix, I love this package and I also love using the most recent version of react 😁
Super useful package, keep up the great work 👍

@Paratron any chance this can be merged?

Need that to ... PLease merge it

@benprofitt @fstn or anyone still stuck with this issue, you can use my branch directly from the git repo in the mean time. I use this in my own projets:

npm install hookrouter@pureartisan/hookrouter#master --save

Please merge this so we can start using react 17! :)

@benprofitt @fstn or anyone still stuck with this issue, you can use my branch directly from the git repo in the mean time. I use this in my own projets:

npm install hookrouter@pureartisan/hookrouter#master --save

I tried this command, package.json and package-lock.json are correctly updated, but the files in node_modules/hookrouter only contain a package.json and the README.md, I tried already to delete the folder and the package-lock but I always get the same output.

I tried this command, package.json and package-lock.json are correctly updated, but the files in node_modules/hookrouter only contain a package.json and the README.md, I tried already to delete the folder and the package-lock but I always get the same output.

@alvaradojl It's possible that your global node cache still uses the old package.

Anyhow, I've just gone ahead and published a "patched" version with a few other fixes (open PRs).

npm i @patched/[email protected]

Let me know if you have any issues. You can also view the exact changes that are included here:
master...pureartisan:patched

NOTE: Your imports will ofcourse have to use @patched/hookrouter as the package name.

import { navgiate } from '@patched/hookrouter';

I tried this command, package.json and package-lock.json are correctly updated, but the files in node_modules/hookrouter only contain a package.json and the README.md, I tried already to delete the folder and the package-lock but I always get the same output.

@alvaradojl It's possible that your global node cache still uses the old package.

Anyhow, I've just gone ahead and published a "patched" version with a few other fixes (open PRs).

npm i @patched/[email protected]

Let me know if you have any issues. You can also view the exact changes that are included here:
master...pureartisan:patched

NOTE: Your imports will ofcourse have to use @patched/hookrouter as the package name.

import { navgiate } from '@patched/hookrouter';

@prageeth Thanks for the patch, managed to get it working on React 17. The only problem i got was it missing the TS declarations. I copied them from the main version to a separate .d.ts file in my project, and it seems to work alright like this.

I really liked this approach while it worked, but I can not have all these hacks in a production build and so, I'll be removing it from my project... wasted a lot of hours btw before finding this thread. You should merge your fix @prageeth and stop updating this thread.

I mean at this point @prageeth you should merge. @Patatron last update was 8 months ago

Unfortunately I don’t have permissions to merge any PRs in this repo. Therefore, I’m using my own @patched/hookrouter package. I’ve recently published a few more fixes there. So if anyone wants to continue, please feel free to use it.