-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
docs: update $effect
examples
#15463
Conversation
|
|
Setting |
preview deployment is failing but i suspect it's unrelated |
Updates examples to remove things that could possibly be confusing or misleading
Mostly replaces
console.log
with writes tocanvas
. Users should typically reach for$inspect
instead ofconsole.log
Also replaces the example that was setting
count
insidesetInterval
inside an$effect
. I tried telling a generally advanced user yesterday that you shouldn't set state inside an$effect
and he responded "what are you talking about? that's what one of the very first examples shows..."