-
Notifications
You must be signed in to change notification settings - Fork 279
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
0.2.30 Crashes Hard #285
Comments
Did you include the context-menu.js file? @stueynz can you have a look? |
Not sure if related. When I use 0.2.30 with context-menu.js, I get the following exception:
When I revert back to 0.2.28, everything works again. My apologies if this comment isn't related to the crash OP reported. |
Hi, Same problem here since last update. The angular App won't build, and I have this error in the console: Error: [$injector:modulerr] Failed to instantiate module app due to:
[$injector:modulerr] Failed to instantiate module treeControl due to:
[$injector:modulerr] Failed to instantiate module contextMenu due to:
[$injector:nomod] Module 'contextMenu' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.6.9/$injector/nomod?p0=contextMenu
minErr/<@http://localhost:3043/build/bundle.js:71596:12
module/<@http://localhost:3043/build/bundle.js:73783:17
ensure@http://localhost:3043/build/bundle.js:73704:38
module@http://localhost:3043/build/bundle.js:73781:14
loadModules/<@http://localhost:3043/build/bundle.js:76479:22
forEach@http://localhost:3043/build/bundle.js:71888:11
loadModules@http://localhost:3043/build/bundle.js:76463:5
loadModules/<@http://localhost:3043/build/bundle.js:76481:40
forEach@http://localhost:3043/build/bundle.js:71888:11
loadModule… I've checked, the |
@Zorkeel: You also need to include context-menu.js via <script> tag or add it to your build/pack script. |
@frame: Thanks ! That was the missing part. Works perfectly now. |
Why the extra file instead of integrating into the script? |
And why this breaking change on a patch version increment? |
our huge codebase depends on this and now you broke our build.. |
The readme says that the context menu script is optional if not using the I think the author doesn't understand semver, which is critical information for a JS developer. I hope they educate themselves soon! |
If you don't want to add the context menu, and its js file, add this workaround to your existing module definitions:
And carry on as normal. |
From SemVer spec: |
Can't you just put something like |
Been using this control for several years and really like it. However, upgrading from 0.2.28 to 0.2.30 crashes my app. I'm not getting any feedback in the console. My app simply doesn't load. This is usually indicative of a null reference to an expected object. Wish I had more but I'm on a deadline right now.
Running AngularJS 1.6.9, any browser reproduces the issue.
The text was updated successfully, but these errors were encountered: