Skip to content

chore(dependencies): update symbol-observable to 0.2.4#1663

Merged
gaearon merged 1 commit into
reduxjs:masterfrom
benlesh:update-symbol-observable
Apr 25, 2016
Merged

chore(dependencies): update symbol-observable to 0.2.4#1663
gaearon merged 1 commit into
reduxjs:masterfrom
benlesh:update-symbol-observable

Conversation

@benlesh

@benlesh benlesh commented Apr 25, 2016

Copy link
Copy Markdown
Contributor

This updates symbol-observable to 0.2.4 which resolves edge cases where a user may have another polyfill for Symbol.observable in place that results in redux store as not being identifiable as 'observable'. See https://github.com/blesh/symbol-observable/issues#7 and sindresorhus/is-observable#1 (comment)

This updates symbol-observable to 0.2.4 which resolves edge cases where a user may have another polyfill for Symbol.observable in place that results in redux store as not being identifiable as 'observable'. See https://github.com/blesh/symbol-observable/issues#7
@gaearon gaearon merged commit e998d73 into reduxjs:master Apr 25, 2016
@gaearon

gaearon commented Apr 25, 2016

Copy link
Copy Markdown
Contributor

👍

I won’t cut a release right now, as new npm installs should just work. If there aren’t more fixes this week, I can cut a patch in the end of the week.

@benlesh

benlesh commented Apr 25, 2016

Copy link
Copy Markdown
Contributor Author

@gaearon yeah, this is such a minor issue I can't imagine it requiring an immediate release. Very edge-casey, IMO.

@imranismail

Copy link
Copy Markdown

What does symbol-observable brings to redux? I don't see it documented anywhere yet.

@gaearon

gaearon commented May 11, 2016

Copy link
Copy Markdown
Contributor

What does symbol-observable brings to redux? I don't see it documented anywhere yet.

Yeah, we probably won’t document this extensively because it’s a relatively niche feature. You can read about it in #1632.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants