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

(Edge case) Handling of collaborative projects with complicated ACL #521

Open
1 of 3 tasks
anngvu opened this issue Mar 21, 2023 · 2 comments
Open
1 of 3 tasks

(Edge case) Handling of collaborative projects with complicated ACL #521

anngvu opened this issue Mar 21, 2023 · 2 comments
Labels
minor a low priority item

Comments

@anngvu
Copy link
Collaborator

anngvu commented Mar 21, 2023

Describe
(Reporting here for future design improvements.)

Currently, in order to see and select a folder for annotation, the user first must be able to see to a project by having edit access on the project.

In this tricky case, there is a collaborative project involving two labs, one Owner and the other Contributor. The Contributor lab is granted local edit permissions for specific folders within the project, but they don’t actually have edit access on the entire project (Owner lab would prefer Collaborator lab not to have edit access to other data by default, and local settings on every folder are hard to manage).

When Collaborator lab goes to DCA, they can't see the specific folder that they are collaborating on and want to annotate because they don't even see the project. To resolve this, we did just add them with edit permissions on the project, but this might not be what's really wanted/acceptable in other cases.

I think this is related to an existing suggestion somewhere along the lines of "can we add the curator link a folder directly, and if the person has access they can just do the annotation?"

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Priority (select one)

  • Minor ⬇️
  • Major 📢
  • Critical 🆘

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (if applicable, please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@anngvu anngvu added the minor a low priority item label Mar 21, 2023
@MiekoHash
Copy link

@AmyHeiser for visibility.

@AmyHeiser
Copy link

Thanks @anngvu. For the example you provided, would the user have edit access to the individual folder? you also mentioned that would be hard to manage so want to confirm that this is a desired state with access granted at folder level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor a low priority item
Projects
None yet
Development

No branches or pull requests

3 participants