-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
Angular 9 #1484
Angular 9 #1484
Conversation
kamilkisiela
commented
Mar 27, 2020
•
edited
Loading
edited
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there a problem with ivy enabled?
I saw u explicitly disabled it...
Doing: "enableIvy": false Means: Why: It's not the first time I ship Angular 9 and Ivy. We use Angular 9 and Ivy in our big clients with current apollo-angular (v1.8.0) without any issues. All the Angular 8 libraries are compatible with Angular 9 and Ivy. I do this PR just because you guys want to have Angular 9 in peerDependencies, there's no problem with apollo-angular and ivy. |
Thanks for the explanation! |
@flobacher did you test it? |
I also already use it without issue in an Angular 9 ivy project, and I trust your statement that you don't have any issues with it |
But i can test with this new version, to be sure.. |
Version Tested:
Very nice to see all the install warnings go away. Is there a roadmap for version 2.0? |
There are no changes in code, I just want to know if Angular Compiler does or does not scream :) About the progress. Once the |
Awesome! That's what I wanted to know! I will test the alpha versions some more with mutations etc. and keep you posted if any errors come up. So far everything works the same as on v1.8.0 for me. |
@kamilkisiela I included |
I've upgraded apollo-angular to next and also graphql to ^15.0.0 and started to emit this error: `ERROR in node_modules/apollo-client/core/watchQueryOptions.d.ts:56:32 - error TS2315: Type 'ExecutionResult' is not generic. 56 refetchQueries?: ((result: ExecutionResult) => RefetchQueryDescription) | RefetchQueryDescription; 40 remoteResult: ExecutionResult; 44 }): Promise<ExecutionResult>; |
a714f0d
to
87d9551
Compare