-
Notifications
You must be signed in to change notification settings - Fork 12k
Error adding @angular/pwa
v0.7.0-rc.3
#11640
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
Comments
Possible duplicate: #11465 |
#11465 is a little different. It is using cli@next with pwa@latest (which does indeed through that error, but I think it is expected due to "version mismatch" (in a sense)). This issue is about cli@next + pwa@next, which should be compatible (and cause a different breakage than the one reported in #11465). |
I faced the same problem with the latest CLI.
|
Same here |
Same problem |
Same issues |
Someone kindly look into this asap. Thanks. |
Same issue here as well. |
Same issues. and, another project with PWA:
Error:
|
This is still an issue after the CLI 6.1.1 release. |
Khi tạo |
Yup me too |
_rename(from, to) does not check for from and to (after normalization) are identical. protected _rename(from: Path, to: Path): void {
from = this._toAbsolute(from);
to = this._toAbsolute(to);
** if (from == to) return ** is missing
if (!this._cache.has(from)) {
throw new FileDoesNotExistException(from);
} else if (this._cache.has(to)) {
throw new FileAlreadyExistException(to);
} |
Possibly fixed by #11655. |
its not fixed on version 0.7.1 on #11655 problem still exists with a fresh project and on adding existing projects |
Not sure what you mean. #11655 has not been released yet (and thus not included in 0.7.1) 😕 |
ohh sorry then, i thought its fixed in v0.7.0-rc.3 and it has been released as 0.7.1 |
For anyone looking for a workaround, you may rollback to @angular/cli@~6.0.0 with @angular/pwa@~0.6.0. |
@VincentG85 : Are you sure reverting to 6.0.0 is solving it? Because when I'd tried, it gave another error. |
@batbrain9392 Did you also revert @angular/pwa ? |
It works for me, from #11663 |
@virlliNia : I tried what u specified. Project name in package.json : ng-universal Project name in angular.json : ngUniversal |
@batbrain9392 you also need to use cli v6.0.8. |
@virlliNia after folloeing your suggestion , i also followed @alexandremoore, and then it fixed my issue. |
@batbrain9392 were you able to fix it? Having same issue. |
@bamboozooled : Yeah, I could..
|
This seems to have been fixed (by #11655) and works fine with the beta versions:
|
@gkalpak : Although pwa works, since this version has new changes in |
@batbrain9392: That is totally unrelated to the current issue. Feel free to open a new one, if you think there is something broken with |
@gkalpak The issue is that since around CLI v 6.1.0 various commands that used to work just don't. Now this issue is closed and suggestion is - as I understand it - to make basically Sophie's choice between PWA and Material. Don't get me wrong, I really appreciate CLI. Usually it does very good job and is great help at routine tasks. Just recently way too many frequently used commands and functionality seems to be broken and there is no way in stable releases get all that sweet functionality working :( Sorry about the rant probably just needed to vent the frustration. |
Sorry to hear you are frustrated 😞 Just to be clear, there is no suggestion to choose between PWA and material. Both should work (and if they don't, they need to be fixed). This issue was about If there is also a problem with |
@PeS82 Please also note that the error shown above and caused by |
I know there was no explicit suggestion to choose between PWA and Material. But based on @batbrain9392 comment I understand it that I can get adding PWA working by updating to versions you mentioned but since the Today I spent couple of hours diffing So I decided to move on and add PWA to the new project and work there and found out I can't. And that threw me over the top (if that means in English what I think it means) 😀 It is good to hear that this issue is fixed, however the overall experience lately is really frustrating. Just ignore me, maybe it's the never-ending heat this summer that got me too, switching PC off and going to swim, maybe it will help 😁 |
@batbrain9392 Did not still work. I had to eventually add pwa functionality manually as adding @angular/pwa does not work in my existing app. |
A lot of circular references between issues because of 'duplicate'. It looks like finding similar problem between issues is solution, then issue closed. Please, do not enter wanna be - not tested solutions and information about may be "duplicate issue" and close. When you solve problem, then close all issues about problem. We now have to click between issues without finding solution. |
@b-mi what are you referring to? All issues should point to this one, that got fixed, and is now closed. |
Dispersia, circular references to issues are here in common. This issue is not fixed: During standard installing (not beta versions):
ng add @angular/material
Even, now looks like ng add @angular/material do not install @angular/cdk ERROR in node_modules/@angular/material/autocomplete/typings/autocomplete-trigger.d.ts(8,32): error TS2307: Cannot find module '@angular/cdk/bidi'. |
How are any of those circular, nor do they have to deal with the issue that this issue was meant to address? And are those currently existing issues? Where are the issue number for them? |
This is just the turnaround after reading many slackoverflow and github solution. My cli code- **
** Hope this workaround works for you. My Version at the moment |
Thank you abhishek1716! I've been struggling with this for a couple hours. Working solution for a mess of a problem. |
Dispersia, this issue have reference to issues number 11465, 11666, 11662, 11663, 11655, 11839. Every of this issues have references too. We jump from issue to issue and back, and we see how are they "closed", but the issue was not really solved. No, issue is not solved - I tried it yesterday (without tricks as downgrade and beta versions (because it can cause some another problem - I want to have libraries actual)) and the result was in my prev comment. |
I confirm: it works with
Version info:
|
I also just confirmed that it works with
Although, now I get the following error when trying to add material using
|
I can also report the issue that @reyco1 is experiencing. I suspect it is because of stable vs 'next' but here is my console output:
|
@reyco1 I got no erros when using Snapshot Build |
Still not working. Even downgrading CLI or everything else. Keeps showing "Path 'ngsw-config.json' already exists" and it doesn't... |
Same here on #11853 Angular CLI: 6.2.0-beta.2 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug Report or Feature Request (mark with an
x
)Command (mark with an
x
)Versions
OS: Windows 10
Repro steps
npm install --global @angular/cli@next
ng new test-app
cd test-app
ng add @angular/pwa@next --project=test-app
The log given by the failure
Npm dependencies are updated (including
package.json
andpackage-lock.json
), but theadd
command is not completed (nongsw-config.json
, nomanifest.json
, no changes inindex.html
, etc).The text was updated successfully, but these errors were encountered: