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.

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

Model#call() -> Error: "no method 'reduce' on 'localRoot.set(...).reduce'" #533

Closed
jhamlet opened this issue Sep 8, 2015 · 1 comment
Closed
Labels

Comments

@jhamlet
Copy link
Contributor

jhamlet commented Sep 8, 2015

lib/response/CallResponse.js:229

Looks like the Rx = require("rx/dist/rx") (line 0) is pulling in a rx library without the reduce method.

Other files do Rx = require("rx/dist/rx") && require("rx/dist/rx.aggregates") which will pulls in the aggregate methods (reduce being one of them), and is probably why you are not seeing this error in your tests.

If you never get around to invoking those files that do the double-require, the rx.aggregate functionality is not there.

You might also want to update the README to add alias's for both "rx$" and "rx/dist/rx" path(s) to point to the consumer's preferred rx library.

@jhamlet
Copy link
Contributor Author

jhamlet commented Sep 8, 2015

Looks like lib/response/ModelResponse.js is the actual culprit. Since CallResponse~setCallEnvelope is working of the return value from localRoot.set

@jhamlet jhamlet closed this as completed in 03ea58f Sep 8, 2015
ThePrimeagen added a commit that referenced this issue Sep 8, 2015
Closes #533 by require core rx as well as aggregate functionality
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant