- 
          
 - 
                Notifications
    
You must be signed in to change notification settings  - Fork 4.7k
 
          docs: update $effect examples
          #15463
        
          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
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.logwith writes tocanvas. Users should typically reach for$inspectinstead ofconsole.logAlso replaces the example that was setting
countinsidesetIntervalinside an$effect. I tried telling a generally advanced user yesterday that you shouldn't set state inside an$effectand he responded "what are you talking about? that's what one of the very first examples shows..."