Skip to content

Accidental mutation of old model causes invalid JSON Patch #801

Closed
@rmorshea

Description

@rmorshea

Current Situation

Unfortunately, the monkey patch applied to jsonpatch doesn't seem to have really fixed the problem. I don't believe that we'll be able to resolve this issue and, even if we were, the library doesn't really seem to be maintained.

Proposed Actions

Instead of actually computing a diff, just use a replace operation where the value is the new render. This will probably have a performance cost in cases where network latency dominates. The alternative though, is getting an error on occasion when JSON patch fails to generate a correct diff.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions