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

Question :: Can you propose edits to the json-crdt.json spec? #763

Open
sgpinkus opened this issue Nov 11, 2024 · 3 comments
Open

Question :: Can you propose edits to the json-crdt.json spec? #763

sgpinkus opened this issue Nov 11, 2024 · 3 comments

Comments

@sgpinkus
Copy link

Sorry for stupid question, couldn't find a link to a repo under https://jsonjoy.com/specs/json-crdt, or anything about contributing to the spec.

@streamich
Copy link
Owner

I have been collecting spec improvement ideas here: #228

You could add a comment there.

@sgpinkus
Copy link
Author

I was just mainly thinking about typos and maybe missing info. For example https://jsonjoy.com/specs/json-crdt/model-document/crdt-algorithms#Replicated-Growable-Array-(RGA)-CRDT-Algorithm, states the algo takes three params node, ref, elems, but how does the element array get its seed ID? I assumed model.apply() grabs it from the clock but it should still be stated here, because it's important to how the algo works. And a couple of other minor grammatical errors I noticed as I was reading.

@streamich
Copy link
Owner

Thank you for looking into this! Unfortunately, the underlying specification Markdown files are currently part of the site repo, which is not public. I will extract the docs out of there some day and place them in a public repo.

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