react-solid-state icon indicating copy to clipboard operation
react-solid-state copied to clipboard

Expected 2 arguments on createEffect

Open grappleware opened this issue 4 years ago • 1 comments

TypeScript complains that createEffect should have 2 arguments. I believe the following should work without issue:

const [someSignalValue, setSomeSignalValue] = createSignal("foo");

createEffect(() => { 
  console.log(someSignalValue());  // will log "foo"
});

However, the type definition says that there should be a second argument for a value. I'm not sure why this second argument is required or even what it does. Is it a dependency? And if so, shouldn't it be optional since someSignalValue() is automatically tracked?

Maybe I'm misunderstanding. Thanks!

grappleware avatar Aug 22 '21 22:08 grappleware

I didn't actually do the typing for this repo, it was community driven and admittedly I haven't used this in a TS based project yet. If it has a type error with the 2nd argument missing that sounds like a bug with the types. The signatures should really be exporting same types as the main Solid repo.

ryansolid avatar Aug 23 '21 01:08 ryansolid