Skip to content

Correct when derived signal functions run - #1647

Merged
LadyBluenotes merged 4 commits into
solidjs:mainfrom
Janani-bn:docs/derived-signals-692
Oct 10, 2026
Merged

LadyBluenotes merged 4 commits into
solidjs:mainfrom
Janani-bn:docs/derived-signals-692

Conversation

@Janani-bn

Copy link
Copy Markdown
Contributor

Closes #692

Derived signals are plain functions, so changing a dependency does not call them by itself. This updates the guide to explain that the caller's tracking scope is what reruns, adds a one-time read versus createEffect example, and clarifies component-body reads and uncached calculations versus createMemo.

Checked the MDX compiles, Prettier, and git diff --check. Runtime assertions with Solid 1.9.14 verify manual reads do not rerun when count changes, while an effect does. Full site build and rendered-page checks have not been run.

Copilot AI balanced review requested due to automatic review settings September 30, 2026 11:50
@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 26e096c
🔍 Latest deploy log https://app.netlify.com/projects/solid-docs/deploys/6ac9a04c1fcbcb00076b8cd0
😎 Deploy Preview https://deploy-preview-1647--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread src/routes/(0)concepts/(2)derived-values/(0)derived-signals.mdx Outdated
@LadyBluenotes
LadyBluenotes merged commit 219fc25 into solidjs:main Oct 10, 2026
10 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.

[Content]: misleading section about when derived-signals are called

3 participants