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

Add update mask to stable REST API patch dag endpoint #10535

Merged
merged 1 commit into from
Aug 25, 2020
Merged

Add update mask to stable REST API patch dag endpoint #10535

merged 1 commit into from
Aug 25, 2020

Conversation

ephraimbuddy
Copy link
Contributor

This PR adds an update mask to patch dag endpoint.


^ 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.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Aug 24, 2020
@mik-laj mik-laj merged commit d6ce8c8 into apache:master Aug 25, 2020
@mik-laj
Copy link
Member

mik-laj commented Aug 25, 2020

Cool. I still think that it is worth adding a description of the update_mask parameter in the API introduction. Would you like to take care of it?

@ephraimbuddy ephraimbuddy deleted the patch-dag-update-mask branch August 25, 2020 11:13
@ephraimbuddy
Copy link
Contributor Author

Cool. I still think that it is worth adding a description of the update_mask parameter in the API introduction. Would you like to take care of it?

Ok. I will create a PR on it tonight :)

@mik-laj
Copy link
Member

mik-laj commented Aug 25, 2020

@ephraimbuddy I added introduction to API, so your change will be much easier now.

@ephraimbuddy
Copy link
Contributor Author

That's awesome. Currently resolving a lot of conflicts in the other PR about unifying API errors but I have pushed the update mask change, haven't created the PR yet. Will create it this night

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants