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

makes friendly error message for Cannot read property 'attrToRemove' of undefined #607

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdf22
Copy link

@mdf22 mdf22 commented Mar 26, 2018

TypeError: Cannot read property 'attrToRemove' of undefined
at ViewCompiler._compileElement (aurelia-templating.js)

TypeError: Cannot read property 'attrToRemove' of undefined
    at ViewCompiler._compileElement (aurelia-templating.js)
@CLAassistant
Copy link

CLAassistant commented Mar 26, 2018

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Alexander-Taran
Copy link

Did you experience the error without it?
or is it just for the sake of completeness ?

@mdf22
Copy link
Author

mdf22 commented Mar 26, 2018

yes, I did

@Alexander-Taran
Copy link

Please share how
Would be nice to have a separate issue linked to pull request with a description of the issue.
As it is now - It is not obvious what you encountered.

@mdf22
Copy link
Author

mdf22 commented Mar 26, 2018

After udate "aurelia-ui-framework": "4.0.0-Beta.12" to "aurelia-ui-framework": "4.0.0-Beta.15" aurelia stops working with error
bluebird.js:5256 TypeError: Cannot read property 'attrToRemove' of undefined
at ViewCompiler._compileElement (aurelia-templating.js:2801)
main problem,where is error
Correction helps to find an error,maybe more correct if to put a throw instead console.log

@Alexander-Taran
Copy link

@mdf22 check this issue
aurelia/ui-virtualization#109

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.

3 participants