YousefED / SyncedStore

SyncedStore CRDT is an easy-to-use library for building live, collaborative applications that sync automatically.
https://syncedstore.org
MIT License
1.71k stars 51 forks source link

Svelte docs + sandpack #51

Closed YousefED closed 2 years ago

YousefED commented 2 years ago

@SkaterDad I was able to fix the sandpack issue you ran into. After some debugging I found the trick was to add an explicit "svelte" dependency in the package.json. I think this is either a bug in a new version of Svelte, or a sandpack bug. Anyway, the workaround works for now.

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1776967141

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Totals Coverage Status
Change from base Build 1776650404: 0.0%
Covered Lines: 533
Relevant Lines: 626

💛 - Coveralls
SkaterDad commented 2 years ago

Interesting. I hadn't tried specifying an older Svelte version.

I tried svelte: ^3 and svelte:latest to no avail. The rest of the code looks pretty similar to what I had attempted.

Can you explain the significance of putting fragment: "xml in the SyncedStore init?

I just joined your Discord as SkaterDad#5213.

As for promotion in the Svelte community, the Svelte Society Twitter account seems to retweet lots of neat stuff. There's a Made With Svelte site as well that promotes projects regularly.

YousefED commented 2 years ago

Can you explain the significance of putting fragment: "xml in the SyncedStore init?

Removed! Is only needed for the rich-text demo, but that doesn't use svelte atm