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

Fix error when using splattributes but no attributes were passed #68

Merged
merged 1 commit into from
May 8, 2019

Conversation

simonihmig
Copy link
Contributor

Fixes #67

@simonihmig
Copy link
Contributor Author

Seems there was already a test case for this case, but for passing to an element: https://github.com/rwjblue/ember-angle-bracket-invocation-polyfill/blob/master/tests/integration/components/angle-bracket-invocation-test.js#L442-L451

The test added here for a component was failing with that #67 error.

Copy link
Member

@rwjblue rwjblue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work, thank you!

I'll try to merge + release once back at my computer (please remind me if not done in a day or so :P )...

@simonihmig
Copy link
Contributor Author

@rwjblue gentle reminder! 😬

Also, would be awesome if you could have a look again some time at the other issue, specifically #65 (comment).

@rwjblue rwjblue merged commit bcd306f into ember-polyfills:master May 8, 2019
@rwjblue rwjblue added the bug Something isn't working label May 8, 2019
@rwjblue rwjblue changed the title Fix unused splattributes being passed to component Fix error when using splattributes but no attributes were passed May 8, 2019
@simonihmig simonihmig deleted the fix-no-value branch May 8, 2019 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: Cannot convert undefined or null to object
2 participants