Unnecessary peer dependency on react-router-dom

[fdc-viktor-luft]

Please remove the dependency react-router-dom completely from your project.

The only import in your source code that points to react-router-dom is actually part of react-router.

In your tests you can easily replace the imported NavLink with an "a" tag. I have a PR ready, but you're currently not allowing to provide PRs.

And please don't forget to remove the corresponding peer dependency in your package.json.

[icd2k3]

Hey @fdc-viktor-luft, I'll look into it!

PRs should be allowed, you can't submit one?

[icd2k3]

Dependency has been removed in 3.2.8

[fdc-viktor-luft]

Maybe I did something wrong when pushing my PR. But your PR looked very similar to mine. The only left differences:

• In your package.json you forgot to remove the dev dependency "@types/react-router-dom"
• In your rollup.config.js you can remove "react-router-dom" from your globals

But nevertheless thank you for your fast response and fix 👍

[icd2k3]

Thanks @fdc-viktor-luft! I'll remove those other instances shortly