-
Notifications
You must be signed in to change notification settings - Fork 14.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
[AIRFLOW-6381] Remove styling based on DAG id from DAGs page #6985
Conversation
Something is wrong with this change. It contains a huge amount of commit. Can you fix it? |
I have updated the base so should be fine now. |
@@ -88,7 +88,7 @@ <h2>DAGs</h2> | |||
|
|||
<!-- Column 4: Dag Schedule --> | |||
<td> | |||
<a class="label label-default schedule {{ dag.dag_id }}" href="{{ url_for('DagRunModelView.list') }}?_flt_3_dag_id={{ dag.dag_id }}"> |
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.
@mik-laj Do you know if we have custom CSS somewhere based on DAG ID
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.
Don't think we do as dag_id can be literally anything!
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.
I've seen it before, but I believed it had a purpose. :-D
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.
@tigerjz32 Thanks for raising this PR
Waiting for the CI to pass before I merge |
Thank you @kaxil! |
Awesome work, congrats on your first merged pull request! |
Issue link: AIRFLOW-6381
[AIRFLOW-XXXX]
for document-only changesIn case of fundamental code change, 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 UPDATING.md.
Read the Pull Request Guidelines for more information.