josephg / ShareJS

Collaborative editing in any app
Other
4.97k stars 454 forks source link

Use JSON-Patch instead of current format for JSON #121

Open tianshuo opened 12 years ago

tianshuo commented 12 years ago

http://tools.ietf.org/html/draft-ietf-appsawg-json-patch-03

This will make it more compatible with other frameworks

wmertens commented 12 years ago

Myeah, interesting, but:

So right now I don't see much value here... Anybody else?

tianshuo commented 12 years ago

I'm really interested in the json move... this could be better animated than deleting then adding

On Thu, Sep 6, 2012 at 3:32 PM, wmertens notifications@github.com wrote:

Myeah, interesting, but:

  • it's still in draft
  • it's more verbose than what we have
  • it doesn't express all our JSON operations, especially string insert/delete

So right now I don't see much value here... Anybody else?

— Reply to this email directly or view it on GitHubhttps://github.com/josephg/ShareJS/issues/121#issuecomment-8324625.

胡天硕 TiansHUo Blog: http://tianshuohu.diandian.com

wmertens commented 12 years ago

Hmmm, not sure what move means in terms of OT vs del+add. @josephg, @nornagon, thoughts?

nornagon commented 12 years ago

We had a move at one point, and it was really hard to get right. We figured the list move would be enough, so we decided to do away with other kinds of moves.