You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Question prompted by @ajaychandran's suggestion in another issue / pr to make this part of the API smoother.
The methods EventStream.fromSeq, EventStream.fromValue and EventStream.fromTry currently require you to specify this parameter. It's not optional because originally I wasn't sure what the default should be. My impression right now is that emitOnce = false is the most reasonable default.
So I'm inclined to add a default value for this paramtere, emitOnce = false, or, if people are actually using emitOnce = true for good reasons, to provide separate fromSeqOnce / fromValueOnce / fromTryOnce methods.
The text was updated successfully, but these errors were encountered:
I am slightly inclined to the the change with default parameter.
ajaychandran
changed the title
Airstream ysers, do you use emitOnce = true / false? How much and why?
Airstream users, do you use emitOnce = true / false? How much and why?
Dec 30, 2020
Question prompted by @ajaychandran's suggestion in another issue / pr to make this part of the API smoother.
The methods
EventStream.fromSeq
,EventStream.fromValue
andEventStream.fromTry
currently require you to specify this parameter. It's not optional because originally I wasn't sure what the default should be. My impression right now is thatemitOnce = false
is the most reasonable default.So I'm inclined to add a default value for this paramtere,
emitOnce = false
, or, if people are actually usingemitOnce = true
for good reasons, to provide separatefromSeqOnce
/fromValueOnce
/fromTryOnce
methods.The text was updated successfully, but these errors were encountered: