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

Put response deserialization into SerializationPolicy using legacy serializer #97

Closed
ghost opened this issue May 21, 2018 · 2 comments · Fixed by #141, #143, Azure/ms-rest-azure-js#33, #144 or Azure/ms-rest-azure-js#35

Comments

@ghost
Copy link

ghost commented May 21, 2018

No description provided.

@RikkiGibson
Copy link
Member

This might be needed, in addition to response parsing, in order to make it reasonable for our users to implement their own HttpClient.

@RikkiGibson
Copy link
Member

If we add options or a separate factory method to include the xml parser/builder, we could make xml an a-la-carte feature in the bundle as well.

@ghost ghost modified the milestones: Sprint-119, Sprint-120 Jun 11, 2018
@ghost ghost added the Planned-Sprint-120 label Jun 11, 2018
@ghost ghost modified the milestones: Sprint-120, Sprint-119 Jun 14, 2018
@ghost ghost added in progress and removed to do labels Jun 14, 2018
@ghost ghost modified the milestones: Sprint-119, Sprint-120 Jun 15, 2018
@ghost ghost changed the title Put response serialization into SerializationPolicy using legacy serializer Put response deserialization into SerializationPolicy using legacy serializer Jun 18, 2018
@ghost ghost closed this as completed in #141 Jun 21, 2018
@ghost ghost removed the in progress label Jun 21, 2018
@ghost ghost reopened this Jun 21, 2018
@ghost ghost added the in progress label Jun 21, 2018
@ghost ghost removed the in progress label Jun 21, 2018
@ghost ghost reopened this Jun 21, 2018
@ghost ghost added the in progress label Jun 21, 2018
@ghost ghost removed the in progress label Jun 21, 2018
@ghost ghost reopened this Jun 21, 2018
@ghost ghost closed this as completed in #149 Jun 25, 2018
@ghost ghost removed the in progress label Jun 25, 2018
@ghost ghost reopened this Jun 25, 2018
@ghost ghost added the in progress label Jun 25, 2018
@ghost ghost removed the in progress label Jun 25, 2018
@ghost ghost reopened this Jun 25, 2018
@ghost ghost added the in progress label Jun 25, 2018
@ghost ghost added Started-Sprint-120 and removed in progress labels Jun 25, 2018
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment