Closed kxzhang closed 8 years ago
Thanks! All but one of my comments are about moving code from views.py
to manager.py
. Maintaining that separation helps insure that when we do add/remove/list collaborators, the the program does the correct permissions checks, and the postgres dbs are updated correctly.
I'd also like to see a cron job syncing the collaborator grants with the django collaborators table... but that can happen after this pull request. (I'm working on the same problem, trying to delete accounts).
Adding in changes to allow users to see repos that they are made collaborators of.