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.

Sign up for GitHub

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

Jump to bottom

Support PATCH as part of a bundled request (batch and transaction) #1525

Closed
kathipavan opened this issue Sep 25, 2020 · 2 comments
Closed

Support PATCH as part of a bundled request (batch and transaction) #1525

kathipavan opened this issue Sep 25, 2020 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@kathipavan
Copy link
Contributor

kathipavan commented Sep 25, 2020 •
edited
Loading

Hi Team,
Could you please enhance FHIR server to support patch as part of a bundled request (batch and transaction).

Thank you,
Pavan.
@kathipavan kathipavan changed the title Enhancement Request to support patch as part of a bundled request (batch or transaction) Enhancement Request to support patch as part of a bundled request (batch and transaction) Sep 25, 2020
@prb112 prb112 changed the title Enhancement Request to support patch as part of a bundled request (batch and transaction) Support patch as part of a bundled request (batch and transaction) Sep 25, 2020
@prb112 prb112 changed the title Support patch as part of a bundled request (batch and transaction) Support PATCH as part of a bundled request (batch and transaction) Sep 25, 2020
@prb112 prb112 added the enhancement New feature or request label Sep 25, 2020
@lmsurpre
Copy link
Member

lmsurpre commented Feb 1, 2021

Specifically, I think its FHIRRestHelper.processEntriesForMethod that will need to be updated to support PATCH.
Outside of batch/transaction, we support JSON Patch and FHIRPath Patch (not XML Patch). JSON Patch could maybe be supported through passing of Binary resources within the bundle, but honestly I’d probably just focus on FHIRPath Patch instead (assuming it meets the needs of your project).

@prb112 prb112 linked a pull request Feb 5, 2021 that will close this issue
issue-1525 -updated FHIRRestHelper #1917
Closed
@kathipavan
Copy link
Contributor Author

The feature is working fine in dev environment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

issue-1525 -updated FHIRRestHelper Mani-Chandu-ihe/FHIR
3 participants
@prb112 @lmsurpre @kathipavan