-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Flow.concatWith #1168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Could you please provide a use-case for this operator similarly to how you did in #437? |
I think that an use case can be something like that: Processing a document where for each page its emitted a non-terminal state like "InfoNotFound". I did something like that to process a pdf with 10 pages until find a barcode or process the entire pdf.
where
|
If the one is
|
All transformations are rewritten via (unsafe) transforms, but all of (safe) transform, onStart and onCompletion operators collectors are safe. Fixes #1168
In a similar vein to |
* All "emitting" operators (onStart, transform, onCompletion) are moved to Emitter * All transformations in Transform.kt are rewritten via (unsafe) transforms, but all of (safe) transform, onStart and onCompletion operators collectors are safe. * Added migration for startWith and concatWith. * Consistent docs for all migration functions. * JvmMultifileClass for Migration.kt so that renamed/deprecated functions (when they moved there) continue to resolve. Fixes #1168
* All "emitting" operators (onStart, transform, onCompletion) are moved to Emitter * All transformations in Transform.kt are rewritten via (unsafe) transforms, but all of (safe) transform, onStart and onCompletion operators collectors are safe. * Added migration for startWith and concatWith. * Consistent docs for all migration functions. * JvmMultifileClass for Migration.kt so that renamed/deprecated functions (when they moved there) continue to resolve. Fixes #1168
I often times need to concat two flows. In RxJava this is called
concatWith
.A implementation could look like:
The text was updated successfully, but these errors were encountered: