bcgov / trustdidweb

Trust DID Web (did:tdw)
https://bcgov.github.io/trustdidweb/
Apache License 2.0
12 stars 6 forks source link

Drop Patch support #64

Open TelegramSam opened 4 days ago

TelegramSam commented 4 days ago

I know this has been discussed elsewhere, but adding issue for clarity.

I think the complexity of patch isn't worth the benefits it brings. If each entry is just a new full DID Document, it is a much simpler solution.

brianorwhatever commented 3 days ago

Yeah we have been discussing this as we are noticing patches aren't actually less data in the examples we have used. We think maybe our jsonpatch implementations aren't the most performant and could be doing things a bit smarter. We also want to run tests with large diddocs to see the comparisons.. more testing needed.