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

Public Users can still view records/locations and records/parties #1384

Closed
linzjax opened this issue Apr 4, 2017 · 2 comments
Closed

Public Users can still view records/locations and records/parties #1384

linzjax opened this issue Apr 4, 2017 · 2 comments

Comments

@linzjax
Copy link
Contributor

linzjax commented Apr 4, 2017

Steps to reproduce the error

Add a user to an organization and assign them as a Public User for a project.
Login as that user and go to that project page.
Add /records/locations/ to the URL.
::or::
Add /records/parties/ to the URL

Actual behavior

They see a stripped down version of the "map" page.
They see a list of party names and types.
They don't have access to any details.

Expected behavior

I don't think /records/location/ is much of an issue because it's not showing you any information that you don't already have access to. But the party table should be stripped the same way that the resource table is.

@oliverroick
Copy link
Member

This is expected behaviour; there's a discussion about this in #961.

The behaviour is the way it is because we wanted to make listing parties consistent with listing location (or rather putting them on the map). We said you could see locations on the map, but you can't access location details if you're a public user. For parties it's the same, you know they are there but can't access them.

I do agree though that this is confusing. Why do we show people things if you can't access them? I think this should be a binary decision, you can either see it or you can't. But refering back to yesterday's discussion in Slack the decision is not that easy as I'd like to think.

@linzjax
Copy link
Contributor Author

linzjax commented Apr 5, 2017

Ah fair enough. It just felt like I had found "secret" pages since there wasn't a way to navigate to these pages in through the UI, but if this has all been discussed before I will go ahead and close this.

@linzjax linzjax closed this as completed Apr 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants