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

Support yjs in dexie-cloud-addon #2045

Open
wants to merge 119 commits into
base: master
Choose a base branch
from
Open

Conversation

dfahlander
Copy link
Collaborator

No description provided.

dfahlander and others added 30 commits June 10, 2024 19:39

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Still need to think it through. Might need to find a better way than iterating all updates in entire DB periodically.
Could not take latest as it generates double types. Found out we needed version 9.3.1 exactly.
* Renamed updTable to updatesTable
* Renamed id / rowId to parentId
* To lookup whether a certain document is open and find it
* updates
* awareness
* sync
* websocket
…ngs change)
Move all yjs related modules to an yjs directory
* updatesTable is not meaningful outside of client
* server need to know the parent table to verify access
* parent table + yDoc prop combined forms the same uniqueness as updatesTable.
David Fahlander added 8 commits December 5, 2024 09:09
More info in README
David Fahlander added 21 commits January 20, 2025 14:35
…ng dependency of dexie-cloud-common where types such as Invite are defined.
* run triggers instantly when for docs that are not open
* enque triggers while docs are open, so they run when they eventually close

This is a good enough solution for now for dexie starter and the Hackathon even though it's a bit too opinionated and should definitely be revised. The whole triggering API should be revised and there is a better idea to be implented later.
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

Successfully merging this pull request may close these issues.

None yet

1 participant