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

Only show the link for the Users list page to SUs #205

Closed
seav opened this issue May 20, 2016 · 1 comment
Closed

Only show the link for the Users list page to SUs #205

seav opened this issue May 20, 2016 · 1 comment

Comments

@seav
Copy link
Contributor

seav commented May 20, 2016

This is forked from #188.

Currently, any visitor to the platform whether logged in or not sees a link for the Users list page. But since only superusers have the user.list permission, it makes sense to only show the link to them and not to everyone else. Otherwise, we will get questions on who and how normal users can get the permission to see this list.

@seav seav added this to the Version 0.1 milestone May 20, 2016
@ian-ross ian-ross self-assigned this Jun 18, 2016
ian-ross pushed a commit that referenced this issue Jun 19, 2016
This turned up a problem in django-tutelary that means that the
superuser checks added here slow the platform down a lot.  The tutelary
bug needs to be fixed before these kinds of permissions checks will
really become viable.
@seav seav added the ui/ux label Jun 29, 2016
ian-ross pushed a commit that referenced this issue Jul 7, 2016
This turned up a problem in django-tutelary that means that the
superuser checks added here slow the platform down a lot.  The tutelary
bug needs to be fixed before these kinds of permissions checks will
really become viable.
ian-ross pushed a commit that referenced this issue Jul 7, 2016
This turned up a problem in django-tutelary that means that the
superuser checks added here slow the platform down a lot.  The tutelary
bug needs to be fixed before these kinds of permissions checks will
really become viable.
ian-ross pushed a commit that referenced this issue Jul 8, 2016
This turned up a problem in django-tutelary that means that the
superuser checks added here slow the platform down a lot.  The tutelary
bug needs to be fixed before these kinds of permissions checks will
really become viable.
ian-ross pushed a commit that referenced this issue Jul 8, 2016
This required adding a "superuser check" mixin to many of the view
classes.  That turned up a problem in django-tutelary that meant that
the superuser checks added in the mixin slowed the platform down a lot.
The tutelary bug was fixed and the tutelary version bumped here to take
account of that.
ian-ross pushed a commit that referenced this issue Jul 8, 2016
This required adding a "superuser check" mixin to many of the view
classes.  That turned up a problem in django-tutelary that meant that
the superuser checks added in the mixin slowed the platform down a lot.
The tutelary bug was fixed and the tutelary version bumped here to take
account of that.
ian-ross pushed a commit that referenced this issue Jul 8, 2016
This required adding a "superuser check" mixin to many of the view
classes.  That turned up a problem in django-tutelary that meant that
the superuser checks added in the mixin slowed the platform down a lot.
The tutelary bug was fixed and the tutelary version bumped here to take
account of that.
wonderchook added a commit that referenced this issue Jul 9, 2016
@seav
Copy link
Contributor Author

seav commented Jul 9, 2016

Fixed via #352

@seav seav closed this as completed Jul 9, 2016
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