Skip to content
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

#16692 show cron schedule description in UI #5

Merged
merged 49 commits into from
Dec 11, 2021

Conversation

pateash
Copy link
Owner

@pateash pateash commented Dec 11, 2021


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In 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.

pateash and others added 30 commits June 27, 2021 00:03
Co-authored-by: Ryan Hamilton <ryan@ryanahamilton.com>
Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
@pateash pateash merged this pull request into main Dec 11, 2021
pateash added a commit that referenced this pull request Dec 12, 2021
* Create ap_codeql-analysis.yml

* 16692 - deleted file

* 16692 - changes done

* 16692 - test added

* 16692 - pre-commits fixed

* 16692 - pre-commits fixed

* 16692 - pre-commits fixed

* 16692 - Runs removed from description

* Update airflow/www/templates/airflow/dags.html

Co-authored-by: Ryan Hamilton <ryan@ryanahamilton.com>

* 16692 - pre-commits fixed

* 16692 - using timetable for showing description

* 16692 - pre-commits fixed

* Update airflow/timetables/base.py

Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>

* 16692 - fixed types

* 16692 - used Base class for storing common properties

* 16692 - using cached_property for description

* 16692 - using cached_property for description

* 16692 - merge fix

* 16692 - removed dagbase

* 16692 - removed unnecessary changes

* 16692 - test name fixed

* 16692 - timetable_description column added to dag table.

* 16692 - None timetable description updated

* 16692 - tests added

* 16692 - review comments.

* 16692 - precommit fixes

* 16692 - precommit fixes

* 16692 - fixed test checks

* 16692 - ignoring description computation for 6th param in Cron expression

* 16692 - pre-commit fixed

* Update airflow/timetables/base.py

Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>

* 16692 - documentation added for timetable description

* 16692 - resolving PR comments

* 16692 - precommit fixes

* 16692 - documentation fix

* 16692 - using empty string as a default description

* 16692 - test fixed

* [apache#16692] - migration file updated

Signed-off-by: Ashish Patel <ashishpatel0720@gmail.com>

Co-authored-by: Ryan Hamilton <ryan@ryanahamilton.com>
Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
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.

1 participant