-
Notifications
You must be signed in to change notification settings - Fork 161
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
Error with glimmer2 enabled with ember canary #255
Comments
@rwjblue Where does the behavior that is being monkeypatched by FastBoot live now? |
The two monkeypatches are
I'm not sure where cc @tomdale |
@bantic @chancancode @chadhietala Is the plan of record to maintain a Node-compatible DOMHelper in Glimmer? If so, I think we can safely remove this monkeypatch for newer versions. |
@tomdale yes |
Yes, that is what I was trying to say in #255 (comment):
|
And ember-cli-addon-tests uses ember canary, so running addon tests with fastboot is failing for me. These DOMHelper extensions will likely be fixed later, see: ember-fastboot#255
Per request from @rwjblue.
I tried to use ember-cli-fastboot (beta 8) with the latest ember canary with glimmer enabled by default and we get an error from this line.
My test app is at: https://github.com/josemarluedke/glimmer2-fastboot-try
The logs are:
The text was updated successfully, but these errors were encountered: