Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add array.flat and array.flatMap #321

Closed
nlepage opened this issue Dec 4, 2018 · 1 comment · Fixed by #329
Closed

Add array.flat and array.flatMap #321

nlepage opened this issue Dec 4, 2018 · 1 comment · Fixed by #329
Assignees
@nlepage
Labels
✨ feature
Milestone
2.0

Comments

@nlepage
Copy link
Member

nlepage commented Dec 4, 2018

https://mdn.io/Array/flat
https://mdn.io/Array/flatMap
@nlepage nlepage added this to the 2.0 milestone Dec 4, 2018
@nlepage nlepage self-assigned this Dec 18, 2018
@nlepage
Copy link
Member Author

nlepage commented Dec 18, 2018

@frinyvonnick @EmrysMyrddin I'm having a doubt on this one.
immutadot-lodash doesn't have flatMap and flatten.
I had done this intentionnaly in order to avoid functions that would drastically change data's structure...
This has no sense in a redux store, but there may actually be other use cases.
What do you think ?

@nlepage nlepage mentioned this issue Jan 22, 2019
Merged
nlepage added a commit that referenced this issue Jan 24, 2019
@nlepage
✨ Add flat and flatMap fix #321 (#329)
551e60d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nlepage nlepage

Labels
✨ feature
Projects
None yet
Milestone
2.0
Development

Successfully merging a pull request may close this issue.

Add flat and flatMap zenika-open-source/immutadot
1 participant
@nlepage