-
Notifications
You must be signed in to change notification settings - Fork 235
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
Cannot read property name
of undefined
#2912
Comments
name
of undefined
I already opened an issue in aria-query. |
I got a fix with "pnpm": {
"overrides": {
"aria-query": "5.1.3"
},
} |
So this is a bug in |
Link to the offending line
This is breaking ember-cli's CI as well as the super-rentals-tutorial build. |
Looks like it should be an easy fix. Can we get a failing code sample? |
Conveniently, the tests for the rule are failing with floating dependencies in this repo's own CI https://github.com/ember-template-lint/ember-template-lint/actions/runs/5262654063/jobs/9512048400 |
Let's see if this fixes it: #2914 |
cfr. ember-template-lint/ember-template-lint#2912 remove override when fix is released
cfr. ember-template-lint/ember-template-lint#2912 remove override when fix is released
Ugh, aria-query has now released a 5.3 that appears to undo these changes in 5.2.1. Their changelog literally says not to try using 5.2. This breaks us again. |
Nevermind, false alarm! I accidentally was looking at a version of ember-template-lint before this PR. |
aria-query
released a new package 5.2.1 which is throwing an error forundefined
in elementsRoleMap get method. All the component invocations which areundefined
elementRoles are throwing as errors.https://github.com/A11yance/aria-query/releases/tag/v5.2.1
Since the package.json didn't have a lock, the following error is occurring in template files which has a component invocation
-- error: Cannot read property 'name' of undefined undefined
The text was updated successfully, but these errors were encountered: