Skip to content
This repository has been archived by the owner on Jul 23, 2021. It is now read-only.

zipAll is missing in OrderedSet #155

Open
Methuselah96 opened this issue Oct 17, 2020 · 1 comment
Open

zipAll is missing in OrderedSet #155

Methuselah96 opened this issue Oct 17, 2020 · 1 comment
Labels
documentation Improvements or additions to documentation from-original-repo good first issue Good for newcomers

Comments

@Methuselah96
Copy link

From @absassi on Mon, 30 Sep 2019 15:18:23 GMT

What happened

zipAll is not available for OrderedSet, but it is documented at https://immutable-js.github.io/immutable-js/docs/#/OrderedSet/zipAll

How to reproduce

import { OrderedSet } from 'immutable'
OrderedSet().zipAll(OrderedSet())

The above code throws TypeError: OrderedSet(...).zipAll is not a function.

Other collections documented to support zipAll work as expected, so workarounds like the following are available:

OrderedSet().valueSeq().zipAll(OrderedSet())
OrderedSet().toList().zipAll(OrderedSet())

Copied from original issue: immutable-js#1738

@comatory
Copy link

I have provided maybe a simple/naive fix in #216

I'm just hoping to bring attention to it as I wanted to explore the codebase and get your suggestions first.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation from-original-repo good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants