-
Notifications
You must be signed in to change notification settings - Fork 39
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
[Bug] Error: Ember is not defined (Ember v4.1.1) #103
Comments
I also am experiencing this bug. |
same here, additionally it crashes because of setting Ember.testing = true in preview-head.html generated by ember-cli-storybook/lib/util.js Line 142 in 53c3948
|
I am getting this bug as well after upgrading Ember to 4.1 |
ember 4 upgrade will most likely need to be supported by a major bump in this library. Looking into it over the weekend. |
@gabrielcsapo Is there anything I can do to help with this? I'd be happy to look into things, just have no idea where and how to start. |
I think the issue comes actually from the render within storybook itself, not the this package https://github.com/storybookjs/storybook/blob/next/app/ember/src/client/preview/render.ts#L41 |
This should fix the issue storybookjs/storybook#17843 |
@dbendaou I am getting the same issue on a fresh install of Storybook using . I tried monkey-patching in the fix from storybookjs/storybook#17843 and it didn't appear to work. I was able to track down where the I tried a bunch of different configuration options to change this behavior, but couldn't get it to work. Removing the plugin broke the compilation. Hope that's useful in tracking down the source of the bug. FYI, I'm using Ember 4.2.0. |
This maybe related to using a very outdated version of that babel plugin:
Long story short: I guess that babel plugin must be updated to a recent version. |
So I can confirm that the version of I had multiple versions of I also tried setting Note that the problem does appear to be with either
Finally, there was also a Also, there is this PR which seems to indicate that Happy to try other things on my setup if it's helpful for debugging. |
@gabrielcsapo same here. If theres anything I can do to help please let me know. I just recently updated to ember 4.0 for the addons I wanted to use storybook on - so unless there is a known workaround Im paused for the moment. |
I think we have many issues that needs to be addressed here:
|
I've pushed a MR to upgrade our Ember scenario and test against Ember4, this should help the upgrade #113 |
I created storybookjs/storybook#18028 today after doing a bit of research for it. I was able to locally bypass the Ember is undefined issue. But its a whole process ... We will need a few changes to fix this stuff. |
Set window.Ember in preview-head.html. Closes #103
Component in Storybook page is displayed with an error:
package.json:
Steps to reproduce the behavior
ember install @storybook/ember-cli-storybook
npx sb init
npm run storybook
Environment
The text was updated successfully, but these errors were encountered: