Skip to content
This repository has been archived by the owner on Apr 20, 2018. It is now read-only.

Support for nested objects #855

Closed
wants to merge 3 commits into from
Closed

Support for nested objects #855

wants to merge 3 commits into from

Conversation

xgrommx
Copy link
Contributor

@xgrommx xgrommx commented Aug 13, 2015

Hi @mattpodwysocki. According with my last comment #851 (comment) I tried fix this problem and now this syntax working fine. For example this code:

Rx.Observable.wrap(function*(v) {
    return {
        1: {
            3: yield [
                Rx.Observable.just(20).delay(5000), [
                    Rx.Observable.just(30).delay(2000), Rx.Observable.just(40).delay(3000)
                ]
            ]
        },
        2: yield Promise.resolve(v)
    };
})(1000).subscribe(console.log.bind(console));

and this code:

Rx.Observable.wrap(function*(v) {
    return yield {
        1: {
            3: [
                Rx.Observable.just(20).delay(5000), [
                    Rx.Observable.just(30).delay(2000), Rx.Observable.just(40).delay(3000)
                ]
            ]
        },
        2: Promise.resolve(v)
    };
})(1000).subscribe(console.log.bind(console));

working identically. It's awesome because we can create complicated object which contains observables.

@ghost ghost added the cla-already-signed label Aug 13, 2015
@xgrommx xgrommx mentioned this pull request Aug 13, 2015
@mattpodwysocki
Copy link
Member

@xgrommx can you please remove the dist folder from this so that it can be merged?

@xgrommx
Copy link
Contributor Author

xgrommx commented Aug 14, 2015

@mattpodwysocki Do you mean only spawn.js should be merged?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants