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

Improve error message if observer is missing #1538

Closed
robdodson opened this issue May 14, 2015 · 3 comments · Fixed by #1614
Closed

Improve error message if observer is missing #1538

robdodson opened this issue May 14, 2015 · 3 comments · Fixed by #1614
Assignees
Labels

Comments

@robdodson
Copy link
Contributor

If I define an observer on my properties object but don't actually implement the method, I get a strange error in effects.html: Uncaught TypeError: this[effect.method] is not a function. It's really not clear that the issue is a missing observer, I had to track down @cdata who could point me in the right direction. Any chance to handle and send out a helpful error message in this scenario?

@lrbridge
Copy link

+1

@kevinpschaaf
Copy link
Member

Yes, we'll be doing a "useful warning" sweep and adding warnings for some obvious things. For user errors that don't make the cut for having warnings included in Polymer proper, the work-in-progress polylint tool will be much more exhaustive.

@jmcgdz
Copy link

jmcgdz commented May 24, 2015

+1

kevinpschaaf added a commit that referenced this issue May 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants