-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Interesting Cursor Transform Work by Robert Lord #27
base: master
Are you sure you want to change the base?
Conversation
Amazing find, looks like this handles all the built-in json0 functionality,
including text0, and sub-paths (and tests)!... However it doesn't delegate
to subtypes... I'll try to merge my work with this to get the best of both.
…On Tue, Apr 9, 2019 at 3:02 PM Curran Kelleher ***@***.***> wrote:
Faced with the challenge of implementing presence for text0 strings
embedded deep within json0 documents, I was perusing the json0 forks
network <https://github.com/ottypes/json0/network> and stumbled upon this
quite interesting work by @lord <https://github.com/lord>.
I don't fully understand what the changes here are really doing, but I
have a suspicion it may be an interesting lead. I'm creating a PR so it's
easy to see the changes, and any interested parties can discuss here.
Related:
- Adding presence to ot-json0 #25
<#25> by @houshuang
<https://github.com/houshuang>
- Sync ephemeral presence data #207
<share/sharedb#207> by @gkusiba
------------------------------
You can view, comment on, or merge this pull request online at:
#27
Commit Summary
- Add cursor movement tests
- Pass string cursor transform operations
- Pass object cursor transform operations
- Add list insert delete and movement
- Fix transformCursor to point to correct transformPosition
- Remove subtype test
- Update package to be json00
- Add logo to readme
- v2 add e key on ops for event data
File Changes
- *M* README.md <https://github.com/ottypes/json0/pull/27/files#diff-0>
(17)
- *M* lib/json0.js
<https://github.com/ottypes/json0/pull/27/files#diff-1> (75)
- *M* package.json
<https://github.com/ottypes/json0/pull/27/files#diff-2> (13)
- *M* test/json0.coffee
<https://github.com/ottypes/json0/pull/27/files#diff-3> (61)
Patch Links:
- https://github.com/ottypes/json0/pull/27.patch
- https://github.com/ottypes/json0/pull/27.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#27>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADwhzGFKL-oHrMweRu6MJQjAT2ujRsqks5vfI96gaJpZM4ckcci>
.
--
http://reganmian.net/blog -- Random Stuff that Matters
|
Oh jeez, this was from a long time ago! Happy to answer whatever questions I can about it, but honestly considering how little I remember, you're basically reading the code with the same eyes that I am. |
@lord Thanks for sharing. Do you have any code, or even snippets, of where you were using these functions (generating events etc, displaying in the UI etc)? I'm trying to wrap my head around how this interacts with the ideas from the @Teamwork approach to shared cursors/presence, they also use transformCursors, but they had to fork ShareDB to add a concept of presence, which you did not (I'm assuming?). Thanks! |
Yeeea, I can try to find some code, but no promises. IIRC my concept of presence was represented as JSON objects in the doc itself, which were automatically added and removed when clients joined and left by some Feathers.js hook? |
Faced with the challenge of implementing presence for
text0
strings embedded deep withinjson0
documents, I was perusing the json0 forks network and stumbled upon this quite interesting work by @lord.I don't fully understand what the changes here are really doing, but I have a suspicion it may be an interesting lead. I'm creating a PR so it's easy to see the changes, and any interested parties can discuss here.
Related: