Skip to content

choropleth: when geojson is set, the geojson file from cdn.plot.ly is still loaded first #6767

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

Open
LucaVazz opened this issue Oct 31, 2023 · 3 comments
Assignees
Labels
bug something broken P2 considered for next cycle sev-4 cosmetic

Comments

@LucaVazz
Copy link
Contributor

When using type: choropleth and setting a geojson url, the geojosn file from cdn.plot.ly still is loaded first.
I'd expect that only the specified geojson file is loaded and used.

(This would be useful to e.g. implement a strict CSP on a site using plotly)


Example: https://codepen.io/lucavazz/pen/jOdrWgV
(baed on https://codepen.io/lucavazz/pen/poGbybw / #4267)

var data = [{
   // ...
   geojson: 'https://unpkg.com/[email protected]/dist/world_110m.json' 
   // this url is an example, in practice this would point to a relative assets path
}]

image


Is this intended behavior / is some configuration missing in the example to only use the provided geojson or is this a bug in the implementation?

Thanks in advance :)

@gvwilson
Copy link
Contributor

Hi - we are tidying up stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for a while, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. If you'd like to submit a PR, we'd be happy to prioritize a review, and if it's a request for tech support, please post in our community forum. Thank you - @gvwilson

@LucaVazz
Copy link
Contributor Author

@gvwilson This issue is unfortunately still relevant, could you please re-open it?

@gvwilson
Copy link
Contributor

Thanks @LucaVazz - I'll add it to our backlog, but I don't know when someone will be able to get to it :-(

@gvwilson gvwilson reopened this Jul 12, 2024
@gvwilson gvwilson self-assigned this Jul 12, 2024
@gvwilson gvwilson removed their assignment Aug 2, 2024
@gvwilson gvwilson added the P2 considered for next cycle label Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken P2 considered for next cycle sev-4 cosmetic
Projects
None yet
Development

No branches or pull requests

4 participants