X Tutup
Skip to content

Add aux to ResponseItem for API 1.22#442

Closed
marcust wants to merge 1 commit intodocker-java:masterfrom
marcust:feature/add_aux_response
Closed

Add aux to ResponseItem for API 1.22#442
marcust wants to merge 1 commit intodocker-java:masterfrom
marcust:feature/add_aux_response

Conversation

@marcust
Copy link
Copy Markdown

@marcust marcust commented Jan 28, 2016

As mentioned in Issue #437


This change is Review on Reviewable

marcust pushed a commit to alexec/docker-java-orchestration that referenced this pull request Jan 28, 2016
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please set since javadocs tags

@marcuslinke
Copy link
Copy Markdown
Contributor

@marcust Thanks. Will merge after v3.0.0 is released.

@marcust marcust force-pushed the feature/add_aux_response branch from a3d58c6 to 286731a Compare January 29, 2016 09:27
@marcuslinke marcuslinke added this to the 3.1.0 milestone Jan 30, 2016
@KostyaSha
Copy link
Copy Markdown
Member

@marcuslinke as soon as you end your refactorings i plan resolve #404 with reviewing newer API. Let me know when you end.


@JsonIgnoreProperties(ignoreUnknown = true)
public static class AuxDetail implements Serializable {
private static final long serialVersionUID = -9136704865403084084L;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@KostyaSha
Copy link
Copy Markdown
Member

I will pick this into 1.22 branch. I think i we will keep it open until all tests will be added before real master merge.

@KostyaSha KostyaSha self-assigned this Feb 16, 2016
@KostyaSha KostyaSha mentioned this pull request Feb 17, 2016
@KostyaSha
Copy link
Copy Markdown
Member

Can't reproduce "aux" field on 1.10.2 daemon...

@KostyaSha
Copy link
Copy Markdown
Member

@KostyaSha
Copy link
Copy Markdown
Member

@marcust could you provide some dump of this pull response?

@KostyaSha KostyaSha modified the milestones: 3.0.0, 3.1.0 Mar 10, 2016
@KostyaSha
Copy link
Copy Markdown
Member

Merged, but json sample would be welcome!

@KostyaSha KostyaSha closed this Mar 10, 2016
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.

3 participants

X Tutup