Skip to content

Fix typo in Model.nestRemoting - #2245

Merged
bajtos merged 1 commit into
masterfrom
fix/nest-remoting-with-hooks
Apr 20, 2016
Merged

Fix typo in Model.nestRemoting#2245
bajtos merged 1 commit into
masterfrom
fix/nest-remoting-with-hooks

Conversation

@bajtos

@bajtos bajtos commented Apr 19, 2016

Copy link
Copy Markdown
Member

Prevent apps from crashing when using Model.nestRemoting without { hooks: false } option.

Note that it is not possible to reproduce this bug using our current Mocha test suite, because other tests modify the global state in such way that the bug no longer occurs.

This supersedes #1966 and fixes #1951.

/cc @timneedham

Prevent apps from crashing when using `Model.nestRemoting` without
`{ hooks: false }` option.

Note that it is not possible to reproduce this bug using our current
Mocha test suite, because other tests modify the global state in such
way that the bug no longer occurs.
@bajtos

bajtos commented Apr 19, 2016

Copy link
Copy Markdown
Member Author

@slnode test please

@bajtos
bajtos merged commit 4cd84dc into master Apr 20, 2016
@bajtos
bajtos deleted the fix/nest-remoting-with-hooks branch April 20, 2016 07:34
@bajtos bajtos removed the #review label Apr 20, 2016
bajtos pushed a commit that referenced this pull request Apr 20, 2016
Prevent apps from crashing when using `Model.nestRemoting` without
`{ hooks: false }` option.

Note that it is not possible to reproduce this bug using our current
Mocha test suite, because other tests modify the global state in such
way that the bug no longer occurs.

[back-port of #2245]
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.

nestRemoting() throws TypeError without {hooks:false} hack

1 participant