Skip to content
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 warning in Leaflet 1.8.0 #1325

Merged
merged 1 commit into from
Mar 24, 2022
Merged

fix warning in Leaflet 1.8.0 #1325

merged 1 commit into from
Mar 24, 2022

Conversation

patrickarlt
Copy link
Contributor

When an overlay loads we should remove the error handler not the already removed load handler.

@patrickarlt patrickarlt requested a review from gavinr March 23, 2022 19:18
@gavinr
Copy link
Contributor

gavinr commented Mar 24, 2022

tested: fixes the "listener not found" when using with https://unpkg.com/[email protected]/dist/leaflet.js with L.esri.dynamicMapLayer.

@gavinr gavinr merged commit a950aa2 into master Mar 24, 2022
@gavinr gavinr deleted the leaflet-1-8-beta branch March 24, 2022 01:26
@gavinr
Copy link
Contributor

gavinr commented Apr 8, 2022

This was released in v3.0.8 https://github.com/Esri/esri-leaflet/releases/tag/v3.0.8

jgravois pushed a commit to jgravois/esri-leaflet that referenced this pull request Apr 23, 2022
jgravois pushed a commit to jgravois/esri-leaflet that referenced this pull request Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants