snd297 / braintrust

Automatically exported from code.google.com/p/braintrust
0 stars 0 forks source link

Adding studies to project does not add permissions to users of the project #462

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. As admin, go to work tab, open project
2. Add study to project with users
3. Log out and login as one of the project members

What is the expected output? What do you see instead?

Error message on login saying that not authorized to do something.
This can be prevented by opening the individual studies and adding privileges 
to the users manually.

Also, once added to a project does not update the project permissions for the 
study, or at least the project does not show up in the project permissions list 
in the sharing window.

Original issue reported on code.google.com by JBWagenaar on 22 Nov 2013 at 4:21

GoogleCodeExporter commented 9 years ago
This is currently supported by adding the project group(s) with the acl editor 
for the individual datasets. 

Some notes for the developers:

Having read permissions (not edit permissions) automatically added could be a 
convenience, but one with possibly confusing ramifications: namely, adding a 
member to a project won't necessarily mean that the member can read all of the 
project's datasets. Because I can be a project admin and add a dataset that I 
don't own. Therefore I can't update the dataset permissions. 

Having said that, this issue is referring only to an admin managing a project, 
so the confusing consequences would not happen.

In addition, the other side of this issue is that we'll need to remove to all 
project permissions from a dataset when it is removed from a project. 

Original comment by samd.p...@gmail.com on 22 Nov 2013 at 5:51

GoogleCodeExporter commented 9 years ago

Original comment by samd.p...@gmail.com on 22 Nov 2013 at 7:06