Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Apr 14, 2025

part of #43519.

Since the new UI has its own color setting and we no longer allow this, remove it from here


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@eladkal
Copy link
Contributor

eladkal commented Apr 14, 2025

do we need to mention something in change log / migration tools?
I assume that if users keep this in their local settings it will simply be ignore rather than fail deployments?

@kaxil
Copy link
Member Author

kaxil commented Apr 14, 2025

I assume that if users keep this in their local settings it will simply be ignore rather than fail deployments?

Correct

@kaxil kaxil force-pushed the remove-statecolors branch from ea69e99 to c28db15 Compare April 14, 2025 16:03
@kaxil kaxil force-pushed the remove-statecolors branch from c28db15 to 6135828 Compare April 14, 2025 16:47
@kaxil kaxil force-pushed the remove-statecolors branch from 6135828 to 88a7936 Compare April 14, 2025 16:49
@kaxil kaxil merged commit 647de21 into apache:main Apr 14, 2025
92 of 93 checks passed
@kaxil kaxil deleted the remove-statecolors branch April 14, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants