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

Allow admin to add contributors to collection without team #280

Open
RonaldEnsing opened this issue Mar 6, 2020 · 5 comments
Open

Allow admin to add contributors to collection without team #280

RonaldEnsing opened this issue Mar 6, 2020 · 5 comments

Comments

@RonaldEnsing
Copy link
Contributor

The Settings page of the collection shows the option to remove owners from the collection. There is also an Add Contributors section, but it does not show any existing users that can be added. I followed the procedure as listed here https://singularityhub.github.io/sregistry/docs/accounts/robots as a workaround. However, I would assume that it should be possible to add users in the web interface since a section is shown to add contributors.

@vsoch
Copy link
Member

vsoch commented Mar 6, 2020

It's not super intuitive how it works, here is the interface for future us to see:

image

In order to add anyone, you need to make a team first. Once they are part of the team, then they appear as options there. This is to somewhat protect privacy of users, because if I allowed any random user to search / find any other user in the registry, it seems like a violation of privacy. Instead, joining a team with the user is a statement / declaration of working together, and then those contributors will appear as options. There is some documentation here https://singularityhub.github.io/sregistry/docs/interface in case it helps!

@RonaldEnsing
Copy link
Contributor Author

Ah I see your point, but is that also needed if the admin user wants to add contributors?

@vsoch
Copy link
Member

vsoch commented Mar 9, 2020

The way I think about these registries is that there is an admin / superuser team, but the containers should be maximally owned by the user. I think in your case you have this special "robot user" and you want to add yourself to the collection? It seems like a reasonable need, but since it's a more niche use case I think it would still work for you to make a team including the bot, and then add who is needed. If you want to share the details of your specific use case I'd be happy to help you brainstorm possible solutions.

@RonaldEnsing
Copy link
Contributor Author

I do not have the need to share a collection with multiple teams or the need to have multiple collections owned by one team. So I have a collection that I want to share with different users (including a robot user) and I feel that making a team would only add unnecessary complexity. I would simply like to add individual contributors to the collection. I see your point of privacy, however, as the admin user I think that this should be possible from the web interface. The admin has the access anyway with the manage.py interface.

@vsoch vsoch changed the title Web interface does not allow adding users to collection [feedback wanted] allow admin to add contributors to collection without team Mar 10, 2020
@vsoch vsoch changed the title [feedback wanted] allow admin to add contributors to collection without team Allow admin to add contributors to collection without team Mar 10, 2020
@vsoch
Copy link
Member

vsoch commented Mar 10, 2020

I understand your use case, but I don't feel strongly about adding this niche case at the moment because it goes against the design philosophy I had for Teams in the first place, and it's also not the most pressing issue for Singularity Registry Server to work on, and I only have so many extra hours in a day. I'm happy to leave this open for now, and if many other users come and +1, when my queue clears up it could be addressed in the future. For the time being, I've written up an example script that shows how to use manage.py. https://singularityhub.github.io/sregistry/docs/interface#admin-control-of-teams.

if you feel very strongly about this you are welcome to open a pull request that implements the change, I'd be happy to review it. You would need to refactor the view to have custom rendering logic basic on the user being a superuser, and then on submit, ensuring that the posts for a superuser / regular user both still function.

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

2 participants