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

133 - Add "Powered by Esri" to portal id-referenced VectorBasemapLayer source. #135

Conversation

shawnmgoulet
Copy link
Contributor

This PR addresses #133, adding "Powered by Esri" to a portal id-referenced VectorBasemaplayer.

Copy link
Contributor

@gavinr gavinr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great, but I think we can move this outside of the if/else block, to near the top of the _setupAttribution() function - that way we are simplifying the code a bit.

@shawnmgoulet shawnmgoulet requested a review from gavinr July 15, 2022 21:07
@gavinr gavinr merged commit edcea44 into Esri:master Jul 16, 2022
@gavinr
Copy link
Contributor

gavinr commented Aug 12, 2022

This was released in v3.1.4.

@shawnmgoulet shawnmgoulet deleted the smg/133-powered-by-esri-vectorBasemapLayer-item-id branch August 12, 2022 15:29
gavinr-maps added a commit to gavinr-maps/esri-leaflet-vector that referenced this pull request Dec 3, 2024
… cases

In Esri#135 we added `Util.setEsriAttribution` to add the "Powered by Esri" attribution, but after Esri Leaflet v3.0.10, calling `Util.setEsriAttribution` does not add "Powered by Esri" anymore. Esri Leaflet only adds "Powered by Esri" for dynamic attribution situations. So now for non-dynamic situations, we must add the string ourself.
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