-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Upgrade react-dom-router in Airflow UI #60316
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
Upgrade react-dom-router in Airflow UI #60316
Conversation
|
cc: @ephraimbuddy Just in case there is an RC2 for |
7c4bfbb to
3624e7e
Compare
|
Huh, this is causing a bunch of linting errors. Want to double check? |
|
No idea why my local pre-commits didn't catch this. I just pushed a fix. |
3624e7e to
020ad99
Compare
3cf112c to
24a6bd2
Compare
airflow-core/src/airflow/ui/src/components/DagActions/DeleteDagButton.tsx
Show resolved
Hide resolved
24a6bd2 to
91acd7b
Compare
91acd7b to
f91d7c8
Compare
Backport failed to create: v3-1-test. View the failure log Run details
You can attempt to backport this manually by running: cherry_picker 34bfa82 v3-1-testThis should apply the commit to the v3-1-test branch and leave the commit in conflict state marking After you have resolved the conflicts, you can continue the backport process by running: cherry_picker --continue |
* Upgrade react-dom-router in Airflow UI * Fix CI (cherry picked from commit 34bfa82)
|
Manual backport #60456 |
* Upgrade react-dom-router in Airflow UI * Fix CI
We do not appear to be vulnerable to GHSA-2w69-qvjg-hvjx, but still upgrade for safety, and also to catch up with the
react-router-domversion of other UI components in this repo.