effector / patronum

☄️ Effector operators library delivering modularity and convenience ✨
https://patronum.effector.dev
MIT License
297 stars 43 forks source link

Remove redundant `sample` in `once` #306

Closed kireevmp closed 1 year ago

kireevmp commented 1 year ago

With effector@23 calling a derived Unit is an error, so a sample that guarded that behavior is no longer required.

igorkamyshev commented 1 year ago

However, user without TS won't know that returned event is not callable.

I believe, we should preserve current behavior for such a case.