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.

Sign up for GitHub

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

Jump to bottom

Refactor permissions implementation #14

Closed
cdriehuys opened this issue Jul 4, 2017 · 0 comments
Closed

Refactor permissions implementation #14

cdriehuys opened this issue Jul 4, 2017 · 0 comments
Milestone
v0.2 - Cleanup

Comments

@cdriehuys
Copy link
Member

The current permission system relies on limiting the results of each view's get_queryset method. As of now, this approach seems like it will get unwieldy very quickly. Perhaps we should look into something like DRY Rest Permissions.
@cdriehuys cdriehuys modified the milestone: v0.2 - Cleanup Jul 4, 2017
@cdriehuys cdriehuys changed the title Look into refactoring permissions implementation Refactor permissions implementation Jul 5, 2017
@cdriehuys cdriehuys mentioned this issue Jul 5, 2017
Merged
cdriehuys added a commit that referenced this issue Jul 5, 2017
@cdriehuys
Merge pull request #22 from knowmetools/feature/permissions-refactor
fa32d23 
Refactor permissions system using dry-rest-permissions.

Closes #14
@cdriehuys cdriehuys mentioned this issue Jul 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.2 - Cleanup
Development

No branches or pull requests
1 participant
@cdriehuys