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

Wrong custom script URL (breaks EDC Browser) #286

Closed
dthiex opened this issue May 8, 2023 · 4 comments · Fixed by #287
Closed

Wrong custom script URL (breaks EDC Browser) #286

dthiex opened this issue May 8, 2023 · 4 comments · Fixed by #287
Assignees

Comments

@dthiex
Copy link
Collaborator

dthiex commented May 8, 2023

URL in the collection metadata is not using the correct new endpoint (Follow up from: #284)

URL the Script is stored but the custom script repository: https://custom-scripts.sentinel-hub.com/custom-scripts/copernicus_services/global-land-cover/global_land_cover/script.js

URL in the public collection metadata: https://custom-scripts.sentinel-hub.com/copernicus_services/global_land_cover/global-land-cover/script.js

@jonasViehweger
Copy link
Collaborator

I checked as well and those are all that do not work. However I found two visualizations which do not work in the browser out of the box (population_density and Sea Ice Extent). The example links in the public collections repo do work but not when you navigate there from the edc browser interface.

@jonasViehweger
Copy link
Collaborator

Also, there are quite a few collections in the category Miscellaneous which could be moved to Copernicus Services (HRVPP, CORINE, Global Land Cover)

@dthiex
Copy link
Collaborator Author

dthiex commented May 8, 2023

However I found two visualizations which do not work in the browser out of the box (population_density and Sea Ice Extent).

The population density one seems indeed to be broken (#288) the Sea Ice Extent works with the right date range and location.

Also, there are quite a few collections in the category Miscellaneous which could be moved to Copernicus Services (HRVPP, CORINE, Global Land Cover)

I also noticed this and will create an internal issue. This was sorted correctly at some point and I don't know why it's not anymore.

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 a pull request may close this issue.

2 participants