-
Notifications
You must be signed in to change notification settings - Fork 817
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
Update to rc6 #604
Comments
I have the same issue |
I put in a PR with changes to make it run with RC6 (See comment though in #606 ) |
is this the same error you get ?
? |
You get that error if you have the older code, look at the PR for support for RC6, it fixes that. Though it seems to be failing some checks. |
I did upgrade to 0.13 but still having same issue :( |
The PR has not been push back into the master yet and thus would not show up in the update. |
I see... fyi also tried http://plnkr.co/edit/YX7W20?p=preview and changed to rc.6 and same issue. |
wondering if we have a time table to get it pushed to Master as I am show stopped by this, |
Yea the issue stems from the provide function in the code. Basically in RC5 provide() still worked though it was being moved to {provide:''}, in RC6 the provide() was removed so it breaks. The PR was changing all the provide()'s to the new format. |
As for when the PR will be able to enter master, I have no clue, from what I can see the code is failing on test checks, from a quick look it looks like the testing code was changed in RC6 and is breaking now. I assume that needs fixed before it can be pushed in. |
got it tx! |
I will see if I can cherry pick the fix :/ |
The code that needs changed is in 3 files I believe only. |
be greatly appreciated if you had links to them? |
got it tx!!! |
please up version and push to npm fix on rc.6 |
@born2net done 😺 |
tx!!!! |
is google maps now a module? as directives: [GOOGLE_MAPS_DIRECTIVES], of course wont work. |
I tried to import AgmCoreModule, which seems to do the trick but I do get an error of:
|
AgmCoreModule.forRoot() |
I confirmed that in current release can't set the center
workaround
|
0.14.0 is in matter now On Sep 2, 2016 6:00 PM, "JavaScriptNinja" [email protected] wrote:
|
great for rc6, thanks :) |
I was using LazyMapsAPILoaderConfig in my bootstrapModule() like this:
But then I get this error at "provide":
Any thoughts? |
@thousight you should place this provide section in your modules (AppModule) providers arrray... |
Issue description
Update to rc6.
Right now, I can see at least the following issue in
index.ts
:The text was updated successfully, but these errors were encountered: