-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: upgrade argo-ui
components to latest -- 3.4 backport
#12998
Conversation
A cherry pick of argoproj#11585 to release/3.4 Changes from argoproj#11585: * `archived-workflow-details.tsx` changes to `className` * `yarn.lock` updated, `react-toastify` still pinned to 9.0.3 Co-authored-by: Anton Gilgur <agilgur5@gmail.com> Co-authored-by: Yuan Tang <terrytangyuan@gmail.com> Signed-off-by: Alan Clucas <alan@clucas.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@agilgur5 Could you take a look?
argo-ui
components to latestargo-ui
components to latest (3.4 cherry pick)
argo-ui
components to latest (3.4 cherry pick)argo-ui
components to latest (3.4 backport)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like there was a merge conflict resolution mistake here (see below in-line comment) that updated some devDeps unnecessarily, but it doesn't actually impact anything meaningful. Otherwise changes LGTM.
Checked locally that I produced the same yarn.lock
as is here as well and ran yarn test
and yarn lint
since CI isn't running on the release-3.4
base branch. All made no changes. Generated files LGTM.
"babel-loader": "^8.3.0", | ||
"copy-webpack-plugin": "^5.1.2", | ||
"copyfiles": "^2.4.1", | ||
"file-loader": "^6.2.0", | ||
"glob": "^9.3.2", | ||
"glob": "^10.3.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh right, and then I removed glob
in #11630 anyway as it was unused
argo-ui
components to latest (3.4 backport)argo-ui
components to latest -- 3.4 backport)
argo-ui
components to latest -- 3.4 backport)argo-ui
components to latest -- 3.4 backport
A cherry pick of #11585 to release/3.4
Changes from #11585:
archived-workflow-details.tsx
changes toclassName
yarn.lock
updated,react-toastify
still pinned to 9.0.3Specifically fixes #11448 on 3.4
Fixes #11585 missing from #11648 (comment)
Testing
make start UI=true
runs up cleanly.Built argo-cli image and installed it as the UI in a 3.4.16 install. Ran a workflow and clicked around a lot. I didn't observe anything wrong.
The resolution #11448 has been checked.