argo-ui
argo-ui copied to clipboard
Feature/npm and yarn/react router dom 6.20.0
https://github.com/argoproj/argo-ui/pull/480
dependabot automatically updates the react-router-dom version, but as the version is upgraded 6, the functions used change, causing errors in the build.
So, I changed it to use a function that matches the react-router-dom 16 version.
Note that this a (very) breaking change, so it does affect all downstream UI builds as well which we have to be fairly careful with.
I took care of the last remaining react-router v4 piece in Workflows in https://github.com/argoproj/argo-workflows/pull/11891. I haven't checked all v5 to v6 breaking changes yet though. We'd also probably want to upgrade to v5.1 first, per the upgrade guide
Also not sure if CD is able to upgrade. If not possible yet, they'd have to stay on the version before this gets merged.
Stale pull request message