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

Made the binding processing errors prettier #1633

Conversation

mariusGundersen
Copy link
Contributor

Made the binding processing errors prettier by removing the wrapping anonymous function. This was made possible by making the wrapping function a named function instead of an anonymous function, and then removing it only if it has the right name. This fixes #1632

…anonymous function. This was made possible by making the wrapping function a named function instead of an anonymous function, and then removing it only if it has the right name.
brianmhunt added a commit to brianmhunt/knockout that referenced this pull request Nov 26, 2014
@brianmhunt
Copy link
Member

Thanks @mariusGundersen -- I have incorporated changes along the lines of what you have asked for into the #1628 PR. On the basis that it mostly hits the mark for what you're asking here, I'll close this PR (and related issue), but do let me know if you think I've overlooked or misunderstood anything, or improvements could be made.

Cheers

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.

"Unable to process binding" error message should not contain the wrappnig function
2 participants