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

Disable / hide the charts button if a user doesn't have enough perms on the dataset #1144

Closed
marthamareal opened this issue Aug 5, 2022 · 4 comments
Assignees
Milestone

Comments

@marthamareal
Copy link

Problem:

add a widget - Couldn't add a chart/table/counter - got this error "Please try to select another layer or widget type. The server doesn't provide the needed services for the layer and the widget type selected. Possible causes are: The selected layer is a raster layer; WFS service is not available; The WPS process gs:aggregate is not available"

Cause:
If a user doesn't have "Download" permissions on a layer WFS services are forbidden, and charts rely on WFS.

Solution:
Disable / hide the charts button in case the user doesn't have enough permissions on that layer.

@DavidQuartz DavidQuartz self-assigned this Sep 5, 2022
@allyoucanmap allyoucanmap reopened this Sep 13, 2022
@giohappy giohappy added this to the 4.0.2 milestone Sep 13, 2022
@giohappy
Copy link

@marthamareal please test this on master

@marthamareal
Copy link
Author

This works as expected with map layers on master.

However, the same error is available on choosing layers to add on dashboards and geostories. should we open another issue to address this or the error message is enough in these scenarios?
aa

@giohappy
Copy link

@marthamareal @DavidQuartz does it mean this control is only implemented for maps?

@DavidQuartz
Copy link
Member

DavidQuartz commented Sep 15, 2022

@giohappy yes it's currently only implemented in maps. I wasn't aware the issue could be replicated in other resources.

DavidQuartz added a commit to DavidQuartz/geonode-mapstore-client that referenced this issue Sep 15, 2022
giohappy pushed a commit that referenced this issue Sep 19, 2022
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

No branches or pull requests

4 participants