Skip to content

[AIRFLOW-806] UI should properly ignore DAG doc when it is None#2020

Closed
dhuang wants to merge 1 commit intoapache:masterfrom
dhuang:AIRFLOW-806
Closed

[AIRFLOW-806] UI should properly ignore DAG doc when it is None#2020
dhuang wants to merge 1 commit intoapache:masterfrom
dhuang:AIRFLOW-806

Conversation

@dhuang
Copy link
Copy Markdown
Contributor

@dhuang dhuang commented Jan 25, 2017

Check dag.doc_md before we try to convert it to Markdown.

Dear Airflow Maintainers,

Please accept this PR that addresses the following issues:

Testing Done:

  • Verified in UI with DAG doc set to __doc__, but no docstrings. No error and looks as if dag.doc_md was not set or set to empty string.

Check dag.doc_md before we try to convert it to Markdown.
@artwr
Copy link
Copy Markdown
Contributor

artwr commented Jan 27, 2017

LGTM

@asfgit asfgit closed this in 6060f65 Jan 27, 2017
asfgit pushed a commit that referenced this pull request Jan 27, 2017
Check dag.doc_md before we try to convert it to
Markdown.

Closes #2020 from dhuang/AIRFLOW-806

(cherry picked from commit 6060f65)
Signed-off-by: Bolke de Bruin <bolke@xs4all.nl>
alekstorm pushed a commit to alekstorm/incubator-airflow that referenced this pull request Jun 1, 2017
Check dag.doc_md before we try to convert it to
Markdown.

Closes apache#2020 from dhuang/AIRFLOW-806
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants