Skip to content
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

Presence #29

Closed
curran opened this issue Apr 15, 2019 · 3 comments
Closed

Presence #29

curran opened this issue Apr 15, 2019 · 3 comments

Comments

@curran
Copy link

curran commented Apr 15, 2019

Would an implementation of presence be welcome in this repo?

Here are two implementations that already exist in forks:

@jhchen
Copy link
Contributor

jhchen commented Apr 15, 2019

One looks like a WIP and the other unclear on its state. This seems like in the idea stage so more info would be helpful, such as pros/cons of this approach versus others, use in real life productions systems, etc.

Personally I have never implemented presence in this layer in production and am open to being enlightened but again hard to evaluate with this level of detail.

@curran
Copy link
Author

curran commented Apr 15, 2019

@jhchen Thank you for your response.

For more context on the presence topic in general, see ottypes/docs#29 . In summary, this approach is being used in production at Teamwork, but the work has progressed only in the Teamwork forks (of ShareDB and related repositories), and the presence they are using is implemented in a completely new OT type for rich text - https://github.com/Teamwork/ot-rich-text .

It's true that presence for rich-text is WIP, spearheaded in houshuang#1

@jhchen
Copy link
Contributor

jhchen commented Jun 3, 2019

If this gets more progress and traction from the other ottypes folks I'll take another look.

@jhchen jhchen closed this as completed Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants