Hey devs,
When reading the documentation on this page GeoJsonDataSource - Cesium Documentation
The links to the examples in sandcastle appear to be broken
Could not find Sandcastle gallery item with ID "GeoJSON%20simplestyle.html".
Nothing urgent just something to note and maybe check the other links in the documentation
Cheers
Riley
Hi @rileyhowley , thanks for pointing this out! I opened an issue to track it.
opened 09:24PM - 07 Jan 26 UTC
type - bug
category - doc
category - sandcastle
needs triage
### What happened?
As reported at https://community.cesium.com/t/cesium-documen… tation-links-broken-to-sandcastle/44774.
@jjspace we appear to be doing something to replace the IDs, that works for most Sandcastles. For example,
https://sandcastle.cesium.com/index.html?src=Geometry%20and%20Appearances.html
redirects to
https://sandcastle.cesium.com/index.html?id=geometry-and-appearances
However, for these two, the redirection doesn't work.
https://sandcastle.cesium.com/index.html?src=GeoJSON%2520and%2520TopoJSON.html
https://sandcastle.cesium.com/index.html?src=GeoJSON%2520simplestyle.html
Both report `Could not find Sandcastle gallery item with ID...`
I haven't yet done an exhaustive check, so there could be more.
### Reproduction steps
1. Go to https://sandcastle.cesium.com/index.html?src=GeoJSON%2520simplestyle.html
2. See error message in Sandcastle console
### Sandcastle example
https://sandcastle.cesium.com/index.html?src=GeoJSON%2520simplestyle.html
### Environment
Browser: Chrome
CesiumJS Version: 1.137
Operating System: Windows
1 Like
Hey @rileyhowley , just wanted to follow up on this in case you aren’t following the Github discussion.
It seems these were a problem of a “double encoded” URL id. We have some PRs to fix this in sandcastle and in the docs which should be out next month. In the meantime if you, or anyone else, come across any more links like this you should be able to modify the URL by replacing %2520 with just %20 or a pure space character and it will hopefully load correctly.
Thanks for letting us know!
1 Like