Closed meepeek closed 11 months ago
After posted, I just rethink and it's the problem with ts not accept the type. Is there any way to bypass it ? The error reported is on the library code so I should fix the compiler instead of patching the library.
change "" to null seems to fix it but according to the doc, since "" and null should be different on display so I'm not sure it will works.
Also I encounter another 2 errors after that as follow
Type 'void' is not assignable to type 'string | number | readonly string[]'. TS2322
14 | <input
15 | type="text"
> 16 | value={text} < ----- THIS IS THE 1ST ONE
| ^
17 | placeholder="Type something..."
18 | onChange={(e) => setText(e.target.value)} < ------ THE 2ND ONE SAID setText EXPECTED 0 arugments but get 1
19 | />
Hi @meepeek !
Thanks for reporting this. This made me realize that we should probably have written that example using TS, like we've done with the rest of the examples and the docs. Anyhow, there is a simple way to fix that issue:
replacing line 6 with:
const [textChange$, setText$] = createSignal<string>();
should do it.
I will leave this issue open until we've decided whether we should write the main example with TS or with JS.
Thanks!
Added typescript annotation to getting started
getting start example works in jsx but I cannot convert it to work with tsx for the compiler reports issue with function bind overload. I think this is necessary for nowadays, react and typescript pair is widely use. I guess the problem is about js loose function declaration, but I can be wrong for my inexperience in the code so I would like you guys to consider. The error message is as follow: