Skip to content

fix cannot resolve param at OverlordResource#getTasks#6593

Merged
gianm merged 1 commit intoapache:masterfrom
QiuMM:Fix_OverlordResource
Nov 13, 2018
Merged

fix cannot resolve param at OverlordResource#getTasks#6593
gianm merged 1 commit intoapache:masterfrom
QiuMM:Fix_OverlordResource

Conversation

@QiuMM
Copy link
Copy Markdown
Member

@QiuMM QiuMM commented Nov 9, 2018

There is no a path param for this endpoint, so fixed it.

Copy link
Copy Markdown

@surekhasaharan surekhasaharan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops.. sorry for this. Thanks @QiuMM for fixing it.

@dclim dclim added this to the 0.13.0 milestone Nov 12, 2018
Copy link
Copy Markdown
Contributor

@gianm gianm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

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.

4 participants