-
Notifications
You must be signed in to change notification settings - Fork 106
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
Issue building with angular-cli 1.0.0 #31
Comments
I have the same issue, can't use ng-cli 1.0 due to this bug.
worked fine in release rc.1 |
@rustygreen still having the same issue? |
@born2net, yes I am. I have not found a fix yet. |
@rustygreen Rob helped out, I had to change in my package.json lib dependencies to peerDependencies and fixed the issue. |
Hi @born2net can you specify how your package.json file ended? Mine is
but still reproduce the error |
see here: |
Thanks for your support guys. I moved the angular dependencies to peerDependencies for now and published a new version of angular2-image-gallery to NPM (0.6.2).
If there are any different issues with the new version regarding angular-cli 1.0.0 please raise a new issue for this. I will close this one for now. |
I'm still receiving the same error as @rustygreen even after your 0.6.2 update. ...
"dependencies": {
"@angular/animations": "^4.0.0",
"@angular/common": "^4.0.0",
"@angular/compiler": "^4.0.0",
"@angular/compiler-cli": "^4.0.0",
"@angular/core": "^4.0.0",
"@angular/forms": "^4.0.0",
"@angular/http": "^4.0.0",
"@angular/platform-browser": "^4.0.0",
"@angular/platform-browser-dynamic": "^4.0.0",
"@angular/platform-server": "^4.0.0",
"@angular/router": "^4.0.0",
"angular2-image-gallery": "^0.6.2",
"hammerjs": "^2.0.8",
"typescript": "^2.2.1",
"web-animations-js": "^2.2.2",
"zone.js": "^0.6.23"
},
"devDependencies": {
"@angular/cli": "^1.0.0",
"@types/hammerjs": "^2.0.33",
"@types/jasmine": "2.5.38",
"@types/node": "^6.0.42",
"codelyzer": "~2.0.0-beta.1",
"jasmine-core": "2.5.2",
"jasmine-spec-reporter": "2.5.0",
"karma": "1.2.0",
"karma-chrome-launcher": "^2.0.0",
"karma-cli": "^1.0.1",
"karma-jasmine": "^1.0.2",
"karma-remap-istanbul": "^0.2.1",
"protractor": "4.0.9",
"ts-node": "1.2.1",
"tslint": "^4.0.2",
"typescript": "~2.0.3",
"webdriver-manager": "10.2.5"
}
} Error is here:
|
It seems like we've to do some work for being AOT-compatible: If anyone finds and/or removes the remaining issues in the code or knows of an easy way to find the 'not-aot-compatible' lines of code, please contribute. :) Otherwise I will have a look at this as soon as I'll find some free time. |
I got the entire thing to work by downloading your source code and adding it to my folder structure and then importing each into the app module and declaring/providing the component/service. |
Interesting, so we have a dirty workaround. ;) So it could also be related to my whacky approach to create the library... However, this issue remains and we need to fix it. |
Alright, folks. I changed the library creation process to use the NGC rather than the TSC compiler. With the new approach even AOT and the angular-cli 1.0.0 integration shouldn't be an issue anymore. Could one of you retry with the newly published angular2-image-gallery 0.7.0 version? |
Update for Would love to hear feedback regarding this issue. Could somebody test it? Otherwise I'll close this issue. |
I have issue with v0.7.4
|
@BenjaminBrandmeier ignore my last comment because when I execute this:
Then all errors are gone :D This module is fantastic Photo Gallery, good job! |
@stepanic Thanks for the feedback. :) |
After upgrading to the latest Angular and CLI version, I can no longer build the project when "angular2-image-gallery" is include. I created a brand new project with the new angular-cli 1.0.0 and only added the "angular2-image-gallery" module (and dependencies) - per the README instructions (I have a working instance back under angular-cli version 1.0.0-RC1). I now get the following error ANY time I try to build (regardless of whether I am using AOT or not):
Environment:
OS: Windows 10
Node: 6.9.1
Angular CLI: 1.0.0
Any thoughts? This seems related: angular/angular-cli#3707
I'll post the code if needed. Thanks!
The text was updated successfully, but these errors were encountered: