Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

Describe JsonPatch correctly in ApiExplorer #5464

Closed
Eilon opened this issue Oct 27, 2016 · 2 comments
Closed

Describe JsonPatch correctly in ApiExplorer #5464

Eilon opened this issue Oct 27, 2016 · 2 comments

Comments

@Eilon
Copy link
Member

Eilon commented Oct 27, 2016

In the JsonFormatter package we need to register an IApiDescriptionProvider that mutates parameters of type JsonPatchDocument<T> into being Operation[].

See domaindrivendev/Swashbuckle.AspNetCore#199 for what some current workarounds are. It would be nice if we had this built-in to avoid the need for workarounds.

@jbagga
Copy link
Contributor

jbagga commented Nov 23, 2016

@rynowak Any thoughts? Should the IApiDescriptionProvider be in the ApiExplorer package or Formatters.Json? The patch needs JsonPatchDocument to be applied, so it should be mutated to Operation[] after that?

@rynowak
Copy link
Member

rynowak commented Nov 23, 2016

We should consider moving the IApiDescription and the stuff needed to implement one to abstractions. We should however keep the infrastructure needed to read descriptions in core.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants