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

tldraw using /sync #158

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

tldraw using /sync #158

wants to merge 45 commits into from

Conversation

fnwbr
Copy link
Member

@fnwbr fnwbr commented Mar 19, 2024

This is a Work-In-Progress to embed tldraw, based on the prototype #126. It respects the "human-readable" first approach.


Todos:

  • ‼️ use an interval to send updates to Matrix when dragging/moving shapes, changing text on notes
    [?] promote users to moderator when inviting them (or toggle switch to decide if they should be able to(read/write))
  • if there is a read-only view: disable all functions if a user is read-only
  • add support to migrate schema updates (see Add basic migrations tldraw/tldraw-yjs-example#14 for example)
  • fix the autofocus (aka fix keyboard shortcuts; see https://tldraw.dev/examples/basic/editor-focus)
  • use self-hosted icons instead of unpkg.com
  • hide all features from the UI that are currently not supported, or support them:
  • various @TODO comments in the code

robertschnuell and others added 29 commits January 18, 2024 15:46
# Conflicts:
#	lib/Matrix.js
# Conflicts:
#	components/layouts/partials/navigation.js
#	lib/Matrix.js
#	package-lock.json
#	package.json
…lient that correctly initiates only _once_

But, I hit a roadblock now: Even though I might have an up to date "store" using ImmerJS now, I don't know how to continue to make that store be the single source of truth for tldraw.
@fnwbr fnwbr self-assigned this Mar 19, 2024
pages/tldraw/[[...roomId]].js Dismissed Show dismissed Hide dismissed
components/TldrawEditor.js Fixed Show fixed Hide fixed
components/TldrawEditor.js Fixed Show fixed Hide fixed
@robertschnuell
Copy link
Member

'promote users to moderator when inviting them (or toggle switch to decide if they should be able to(read/write))' has been solved in develop by @aofn -> ff74c4b

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.

3 participants