-
Notifications
You must be signed in to change notification settings - Fork 2
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
getCapabilities improvements #133
Conversation
✅ Deploy Preview for deltares-viewer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for rws-viewer ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
🚀 Deployed on https://67935f5c85610dc095ec384a--nl2120.netlify.app |
🚀 Deployed on https://67935f5df95947c0c36f785c--rws-viewer.netlify.app |
There was an issue with downloading the CDDA data layer with the rws viewer. I figured out that the layer data also has a downloadUrl attribute that is not always populated. The CDDA dataset has a WMS and a WFS service. So the getCapabilities request will work for both service types. This will cause an error when trying to download because the viewer will attempt to download from the WMS url instead of the WFS url. I fixed this by adding a check that checks if there is a downloadUrl attribute in the layer data and use that url to download if it exists. |
No description provided.