-
Notifications
You must be signed in to change notification settings - Fork 104
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
Can't build production dist files with bootstrap-for-ember #168
Comments
@gordonbaz it’s actually this addon that you had installed: https://github.com/dockyard/ember-cli-bootstrap |
I am having the same problem, but don't have the ember-cli addon installed. Just importing "ember-addons.bs_for_ember": "~0.7.0" in my bower.json My project runs fine locally, but when i compile for production I am getting errors. |
@gordonbaz how did you manually use the bootstrap addon? I am having the same problem. |
I changed my brocfile.js to read as follows: app.import('bower_components/bootstrap/dist/js/bootstrap.js'); module.exports = mergeTrees([app.toTree(), extraAssets]); |
@gordonbaz was that a fix/workaround, or is this still broken? Having same issue.. |
I must admit that I haven't tried out ember-cli-bootstrap since putting this fix in place - it's still working for me now though |
just looking at that code, it seems to be pulling in bootstrap - but not the other fun stuff in ember-cli-bootstrap. If you are using their modal shortcuts, and the notification stuff, I don't think that will work.. I'll keep trying :) |
OH - I did get passed my issue though - it had to do with the ordering in package.json, now I have NEW issues! |
i noticed a relation to this issue: ember-cli/ember-cli#972 bootstrap-for-ember/dist/js/bs-core.max.js Line 420 in f20d921
|
Hi there,
I've just had a problem with building production dist files and after lots of debugging (many, many thanks to Jamie White) we tracked down the problem to this ember-cli addon. After removing it and manually adding bootstrap the problem was resolved. See ember-cli/ember-cli#1727 for more detail
Thanks,
Gordon
The text was updated successfully, but these errors were encountered: