-
Notifications
You must be signed in to change notification settings - Fork 997
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
Fix Deprecated Console Output for Leaflet 1.2.0 #772
Conversation
test passing now
Uh did you actually try this? I think you meant to include L.Evented, L.Events, and at any rate, you can't use .include directly on Leaflet.Class objects. You need to use something like, for example:
etc |
@germanjoey yes i tried it and it was working for me. |
Obviously, I tried it and it did not work for me. I had to change it to what I posted. |
Fix Wrong Class
Fix Wrong Class
I'd be real glad if this could be merged. My console is basically useless now |
@ddproxy is there any reason for not accepting this MR ? |
Looks like this happens on Leaflet 1.1.0 as well, but ya I'd love it if this could be merged in? |
Any updates on this PR? How can I help? |
@ddproxy any update on this? |
+1 |
Thanks! |
Sorry for taking so long on this, I'm working up a release for today. |
@ddproxy has this been released? I see the PR is merged, but can't seem to find a release that includes these changes. |
Bump |
Sorry this has gone out to NPM today. |
@ddproxy Thanks! Appreciate everything that you do 💯 |
Fix Deprecated Console Output for Leaflet 1.2.0
This fixes the Deptrecated Messages in Leaflet 1.2.0