harrisjoseph opened a new pull request #10839:
URL: https://github.com/apache/airflow/pull/10839


   This PR populates the **Configuration JSON** text-area in the '**Trigger 
DAG**' UI with the DAG's `param` dict.
   
   Currently the UI looks like this: 
   ![Screenshot 2020-09-09 at 17 25 
29](https://user-images.githubusercontent.com/17148324/92626182-7db75c00-f2c1-11ea-8099-ddebd9730f0a.png)
   
   When a job can be run with many parameters, it may be easier to provide 
users with the default values that they can change.
   
   An artist's rendition of the proposed change:
   ![Screenshot 2020-09-09 at 17 25 
29](https://user-images.githubusercontent.com/17148324/92627265-04b90400-f2c3-11ea-833b-01e5ce076c27.png)
   
   Fortunately for me, the `conf` parameter was already present in the template:
   
https://github.com/apache/airflow/blob/master/airflow/www/templates/airflow/trigger.html#L31
   
   
   I will add / change some tests for this shortly
   
   ---
   
   A minor aside here - I love this addition to the UI, I think I'm going to be 
using it a lot in the very near future. Huge thanks to @dacohen and all of the 
reviewers & maintainers involved in this feature 🙌 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to