Fixed the final warning for Request.publisher to Effect
Update TCA to 1.0.0
Fix nested binding bug
I think the ideal fix is to change the model of Request from publisher to async function, because request should only be evaluated once, but this is not practical to do in one pr.
I found a useful snippet from this artical, which transform a publisher to an async throwing function. I modified this snippet to non-throwing and use a typed error.
I found the nested binding was broken in my earlier pr. There is a suggested fix from version 0.55.0 to use Reducer.onChange
In this pr:
Request.publisher
toEffect
I think the ideal fix is to change the model of
Request
from publisher to async function, because request should only be evaluated once, but this is not practical to do in one pr. I found a useful snippet from this artical, which transform a publisher to an async throwing function. I modified this snippet to non-throwing and use a typed error.I found the nested binding was broken in my earlier pr. There is a suggested fix from version 0.55.0 to use
Reducer.onChange