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

PATCH support #302

Closed
andy-a-o opened this issue Nov 29, 2020 · 5 comments · Fixed by #341
Closed

PATCH support #302

andy-a-o opened this issue Nov 29, 2020 · 5 comments · Fixed by #341

Comments

@andy-a-o
Copy link
Contributor

andy-a-o commented Nov 29, 2020

Is your feature request related to a problem? Please describe.
http://www.hl7.org/fhir/STU3/http.html#patch

Describe the solution you'd like
PATCH method is supported as described in the spec.

Failed tests
ConnectathonPatchTrackTest

@jjrdk
Copy link
Contributor

jjrdk commented Mar 1, 2021

I created a draft PR to propose a way to add support for patch.

@kennethmyhra kennethmyhra linked a pull request Mar 19, 2021 that will close this issue
@jjrdk
Copy link
Contributor

jjrdk commented Apr 8, 2021

@kennethmyhra Can we close this issue now that #341 has been merged?

@kennethmyhra
Copy link
Collaborator

Absolutely

@andy-a-o
Copy link
Contributor Author

http://www.hl7.org/fhir/R4/http.html#patch

The body of a PATCH operation SHALL be either:
a JSON Patch document with a content type of application/json-patch+json
an XML Patch document with a content type of application/xml-patch+xml
a FHIRPath Patch parameters resource with FHIR Content Type

for now, it seems that only FHIR Patch was implemented. Can't really say this issue is solved... Can't re-enable tests because of that (tests send JSON patch).

@andy-a-o
Copy link
Contributor Author

#384

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 a pull request may close this issue.

3 participants