-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
feat(errors): update error messaging to provide solutions #3001
feat(errors): update error messaging to provide solutions #3001
Conversation
Pull Request Test Coverage Report for Build 2858
💛 - Coveralls |
|
||
if (isNil(index)) { | ||
message += `. Please make sure that the "${key.toString()}" property is available in the current context.`; | ||
message += `.\nPlease make sure that the "${key.toString()}" property is available in the current context.`; | ||
return message; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add potential solutions here as well.
(side note: index
is nil
when the property-based injection is used - no constructor. The rest remains the same)
…or-message-updates
…or-message-updates
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
What is the new behavior?
Updated error messaging to provide solutions and pinpoint where the issue is stemming from.
Does this PR introduce a breaking change?
Other information