blaze icon indicating copy to clipboard operation
blaze copied to clipboard

Support PATCH Method

Open MM-Lehmann opened this issue 2 years ago • 4 comments

Judging from the tests I've made, I suspect the PATCH method is not supported in blaze at the moment? Is this planned at any point? What would be an appropriate way of updating resources without transferring the entire resource?

Cheers

MM-Lehmann avatar Nov 04 '22 07:11 MM-Lehmann

Hi Martin,

right, PATCH is not supported. In your use-case, is it really a problem to transfer the whole resource?

alexanderkiel avatar Nov 07 '22 17:11 alexanderkiel

It's not a huge problem but just more work to download, modify and validate the resource instead of simply uploading the patch.

I need to add an identifier to the list of patient identifiers. But maybe there is a more efficient way i am not seeing?

Am 07.11.2022 um 18:06, Alexander Kiel @.***> schrieb:

Hi Martin,

right, PATCH is not supported. In your use-case, is it really a problem to transfer the whole resource?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

MM-Lehmann avatar Nov 07 '22 18:11 MM-Lehmann

It's not a huge problem but just more work to download, modify and validate the resource instead of simply uploading the patch. I need to add an identifier to the list of patient identifiers. But maybe there is a more efficient way i am not seeing?

Validation is a good point here. Because Blaze doesn't validate, downloading, modifying, validating locally and uploading is a good practise, I would say.

I'll leave this issue open, but PATCH has not the highest priority right now.

alexanderkiel avatar Nov 08 '22 14:11 alexanderkiel

preferred with JSON

DWay17 avatar Dec 14 '23 08:12 DWay17