[http] WIP, merge-patch implementation #7121
Conversation
witemple-msft
requested review from
bterlson,
markcowl,
allenjzhang,
timotheeguerin,
chrisradek and
AlitzelMendez
as code owners
|
All changed packages have been documented.
Show changes
|
|
You can try these changes here
|
| if (isMutableType(member.type)) { | ||
| const variant: UnionVariant = { | ||
| ...member, | ||
| type: mutateSubgraph(program, [self], member.type).type, |
There was a problem hiding this comment.
I think we want to apply CreateOrUpdate visibility to variants in the Update case, so feels like the optional mutator should be applied.
| } else if ($(realm).record.is(model) && isMutableType(model.indexer!.value)) { | ||
| clone.indexer = { | ||
| key: model.indexer!.key, | ||
| value: mutateSubgraph(program, [self], model.indexer!.value).type, |
There was a problem hiding this comment.
For record types, we also want to apply CreateOrUpdate, so I think the optional mutator, because any key could exist or not in the resource.
|
Superseded by #7207 |
This work-in-progress PR implements the proposal for JSON merge-patch support in the HTTP core library.
Still to-do: