argo-ui icon indicating copy to clipboard operation
argo-ui copied to clipboard

Feature/npm and yarn/react router dom 6.20.0

Open sunyeongchoi opened this issue 1 year ago • 1 comments

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.

sunyeongchoi avatar Nov 27 '23 01:11 sunyeongchoi

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.

agilgur5 avatar Jan 06 '24 18:01 agilgur5

Stale pull request message

github-actions[bot] avatar Mar 07 '24 11:03 github-actions[bot]