Skip to content

Conversation

@signalize
Copy link
Contributor

resolves #1566

  • This change fixed the error
  • Updated the docs
  • With this change it's possible to do this:
let value = $derived(Posts.find().fetch());

instead of

let items = $state.raw([]);
$effect(() => {
  const cursor = Posts.find({});
  items = cursor.fetch();

  return () => {
    cursor.cleanup();
  };
});

@codecov
Copy link

codecov bot commented Apr 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (fc98b66) to head (88f6e48).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1595   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           56        56           
  Lines         1789      1796    +7     
  Branches       417       418    +1     
=========================================
+ Hits          1789      1796    +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

signalize and others added 2 commits April 4, 2025 12:01
fix: modified the svelte test

docs: update svelte documentation

docs: update svelte documentation
@maxnowack maxnowack enabled auto-merge (rebase) April 4, 2025 10:04
@codecov
Copy link

codecov bot commented Apr 4, 2025

Bundle Report

Changes will increase total bundle size by 326 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@signaldb/svelte 1.01kB 326 bytes (47.45%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: @signaldb/core

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.cjs29.js 539 bytes 815 bytes 195.29% ⚠️
index29.mjs 539 bytes 808 bytes 200.37% ⚠️
index.cjs27.js -52 bytes 763 bytes -6.38%
index27.mjs -52 bytes 756 bytes -6.44%
index.cjs28.js -487 bytes 276 bytes -63.83%
index28.mjs -487 bytes 269 bytes -64.42%
view changes for bundle: @signaldb/svelte

Assets Changed:

Asset Name Size Change Total Size Change (%)
index.svelte.js 286 bytes 805 bytes 55.11% ⚠️
index.svelte.d.ts 40 bytes 208 bytes 23.81% ⚠️

@maxnowack maxnowack merged commit b08dd6f into maxnowack:main Apr 4, 2025
15 checks passed
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.

svelteReactivityAdapter with persistence causes an error

2 participants