Iterate over copy of observers for notifyObservers [Fixes #151]

optimizely / nuclear-js

Reactive Flux built with ImmutableJS data structures. Framework agnostic.

https://optimizely.github.io/nuclear-js/

MIT License

2.23k stars 141 forks source link

Closed c-spencer closed 9 years ago

c-spencer commented 9 years ago

A cleaner alternative to #152 that does not call new observers attached during notification.

bhamodi commented 9 years ago

LGTM. I'll wait on @jordangarcia's approval before merging.

jordangarcia commented 9 years ago

This looks good!

Thanks for doing, apologies for the wait on CR.