Skip to content

Comments

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

Merged
mik-laj merged 1 commit intoapache:masterfrom
ephraimbuddy:patch-dag-update-mask
Aug 25, 2020
Merged

Add update mask to stable REST API patch dag endpoint#10535
mik-laj merged 1 commit intoapache:masterfrom
ephraimbuddy:patch-dag-update-mask

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