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 some issues with Glimmer2 usage. #75

Merged
merged 2 commits into from
Nov 10, 2016

Conversation

rwjblue
Copy link
Contributor

@rwjblue rwjblue commented Nov 8, 2016

There are still a few failing tests against beta/canary/alpha, but this
fixes a number of the failures (due to the difference in return value of
getDOM).

Fixes #74.

@rwjblue rwjblue changed the title Fix _some_ issues with Glimmer2 usage. Fix some issues with Glimmer2 usage. Nov 8, 2016
Robert Jackson added 2 commits November 8, 2016 18:03
Also, update return value from `getDOM` helper to be consistent across
Ember versions (it will always return the `document` now).
@rwjblue
Copy link
Contributor Author

rwjblue commented Nov 8, 2016

FWIW the failing test is the one we knew about (added in #68) and is being referenced in #66.

@rwjblue
Copy link
Contributor Author

rwjblue commented Nov 10, 2016

@lukemelia - When you have a minute, could you take a look?

@krisselden krisselden merged commit 978812f into yapplabs:master Nov 10, 2016
@rwjblue rwjblue deleted the use-document-service-correctly branch November 10, 2016 11:56
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.

Emberjs 2.10.0-beta.2 issues
2 participants