Skip to content

Conversation

@Jocs
Copy link

@Jocs Jocs commented Jan 30, 2016

Hi, mknichel, I really like this article, and have used some examples in my javascript error track project, thank you for your great work on javascript error.
but when i rewrite the Promise.prototype.then method, I found that my '.catch' can not catch error, so I think we should rethrow the error in catch block of 'protectEntryPoint' function.
One more thing, when we use the 'protectEntryPoint' function, we should check the arguments type, and make sure its 'function' type.
Thank you again for you article, I have translated a Chinese version, if you prefer a Chinese version, I'll PULL another request.

Review on Reviewable

@mknichel
Copy link
Owner

mknichel commented Feb 7, 2016

Hi Jocs,

Thanks for the pull request! I left a few comments in the code.

A Chinese translation sounds great - do you want to submit a separate pull request for that?

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.

2 participants