-
Notifications
You must be signed in to change notification settings - Fork 400
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
#9589: Notify data projection not compatible / available for COG #9690
Conversation
@dsuren1 this PR after our recent PRs needs to be upated with master, some conflicts present. |
For the cog test service urls, we have projection defs already defined in Dev so, maybe it's because of that, i'm not able to create a test data that satisfies all cases. However, you can test it locally by removing projection defs and adding this service url to test catalog error and import this map map_cog_notify.json to test warning and TOC |
@ElenaGallo, could you please test this on DEV ? Thank you |
Test passed @dsuren1 please backport. Thanks |
…le for COG (geosolutions-it#9690) (cherry picked from commit 76207ab)
@dsuren1 (FYI @ElenaGallo @offtherailz) I've tested this as well the other issues. In this case I've reverted projDefs definitions from DEV data dir The notification works fine and the layer is not added to the map. Probably it would be better to let the user know why somehow (e.g possibly a link to the related user guide section? I'm open to suggestions). There is anyway what I think is a wrong behavior. Every time the map is loaded the following notification appears independently of what is the COG layer visible in TOC (I've tried visualizing them one by one). This is not correct because the projections are supported for all COG layers in TOC (once needed projectionDefs have been restored in DEV data dir). |
It's because the layer's |
Description
This PR check for projection compatibility and notifies the user with meaningful feedback
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
What is the new behavior?
CATALOG
TOC
NOTIFICATION
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information