Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add the equivalent of concatMap #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

denizcoskun
Copy link

concatMap is a quite useful operator in the Rx world and it was a bit disappointing that Combine didn't have a built-in equivalent of it until I found that flatMap behaves like concatMap when used correctly.

@freak4pc
Copy link
Member

freak4pc commented Aug 8, 2020

I'm not sure that's entirely true, there was a huge discussion about this in one of the slack channels and some folks mentioned it has some differences. I'll try to find that info.

@denizcoskun
Copy link
Author

@freak4pc thanks, really appreciate that!

@denizcoskun
Copy link
Author

I'm not sure that's entirely true, there was a huge discussion about this in one of the slack channels and some folks mentioned it has some differences. I'll try to find that info.

here's where I got the information: https://forums.swift.org/t/combine-what-are-those-multicast-functions-for/26677/26

@freak4pc
Copy link
Member

I couldn't find anything against this claim, if you rebase this I'll merge it . Thanks!

@denizcoskun
Copy link
Author

denizcoskun commented Nov 29, 2020

I couldn't find anything against this claim, if you rebase this I'll merge it . Thanks!

@freak4pc thanks, it is rebased now.

@freak4pc
Copy link
Member

freak4pc commented Dec 1, 2020

Hey @denizcoskun -

There was an interesting research and discussion in CombineExt which seems to indicate this variation isn't quite similar to concatMap in all forms.

What do you think?

CombineCommunity/CombineExt#68

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants