jaegertracing / jaeger-ui

Web UI for Jaeger
http://jaegertracing.io/
Apache License 2.0
1.12k stars 477 forks source link

Upgrade react-router-dom to latest version #1825

Open anshgoyalevil opened 11 months ago

anshgoyalevil commented 11 months ago

This is a Tracker Issue for all react-router-dom related upgrades.

Useful resources:

priyanshu-kun commented 11 months ago

rrdv6 has plenty of breaking changes compare to v4 that currently in use, I think we have to migrate it gradually.

anshgoyalevil commented 11 months ago

Yes, we would have to upgrade it sequentially version by version, keeping the project intact

priyanshu-kun commented 11 months ago

@anshgoyalevil do we have to migrate it to v5 before v6?

anshgoyalevil commented 11 months ago

Yes. First v5, then 5.1, and then v6. (There can be more versions between 5.1 and 6, to which we would have to upgrade first).

Though it is a part of my LFX Mentorship, if you want to help out, you may DM me on Slack, and we can collaborate on another dependency upgrade, so that we both don't have to create the same type of PRs simultaneously.