Recoil icon indicating copy to clipboard operation
Recoil copied to clipboard

Published 20 hours ago • verified publisher icon facebookexperimental

[Proposal] Include Recoil Nexus in the package.

Open luisanton-io opened this issue 2 years ago • 2 comments

As requested by multiple users[1][2][3] and considering the recent traction of Recoil Nexus, hopefully we can include this feature directly in the package.

I am not particularly happy I had to extract RecoilRoot from its original module to avoid a circular dependency with useRecoilCallback. This has led me to rename the exports everywhere from /Recoil_RecoilRoot to /Recoil_RecoilNexus, tests included.

If this is a big deal any hint to avoid doing so is welcome. Cheers, L

luisanton-io avatar May 31 '22 18:05 luisanton-io

Thanks for this contribution @luisanton-io! Though, we should explore some of the nuances of the interface for consideration. It seems like this API would have race conditions in case there are multiple <RecoilRoot>'s used..

drarmstr avatar Jun 07 '22 23:06 drarmstr

I see. Hopefully I will have some time to check into this soon. Thanks for the heads up

luisanton-io avatar Jun 21 '22 07:06 luisanton-io