chore: replace async useComputed$ #11
Conversation
|
| useTask$(async ({ track }) => { | ||
| const src = track(() => props.src); | ||
| const width = track(() => props.width); | ||
| const height = track(() => props.height); | ||
| const aspectRatio = track(() => props.aspectRatio); | ||
| const layout = track(() => props.layout); | ||
|
|
||
| srcSetSig.value = await getSrcSet({ |
There was a problem hiding this comment.
FYI, you can run any sync code you like inside track and track multiple signals at once.
you could so something like await getSrcSet(track(() => props))
There was a problem hiding this comment.
actually I just realized that you need to reference the props themselves, so it would be more like await getSrcSet(track(() => ({src: props.src, width: props.width, ...}) or you could destructure in the component top and just list the props in track.
There was a problem hiding this comment.
I see, would you like to create a PR for that?
Chore to remove warning