Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#181 closed enhancement (fixed)

"Groups this user can admin" API call

Reported by: adehnert Owned by:
Priority: normal Milestone: Fall 2013
Component: Membership confirmation Version:
Keywords: impacts:groups Cc:
Size: medium

Description

kchen points out (-c asa 2012-11-13) that it's silly that https://asa.mit.edu/membership/update/ lists groups that the user, say, has office access for but is not an admin of. That page currently uses groups.models.Group.involved_groups to find what groups to list. groups.models.Group should grow a method that only lists groups the user can admin, and the membership update page should use that method.

Change History (3)

comment:1 Changed 12 years ago by adehnert

  • Keywords impacts:groups added

comment:2 Changed 12 years ago by adehnert

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in commit 7431d1341e577bcca9bdf3ff018cc209ca2bf2a7

Author: Alex Dehnert <adehnert@mit.edu>
Date:   Sun Nov 17 03:55:11 2013 -0500

    Add Group.admin_groups method (ASA-#181)
    
    This adds a Group.admin_groups method, which can be used to find the groups
    that a user is an admin of (where "admin" is defined as "has a specified
    permission [default: admin_group] via an OfficerRole).

comment:3 Changed 12 years ago by adehnert

Err, along with commit 88843ac114162e6392605ae513126b776793a787

Author: Alex Dehnert <adehnert@mit.edu>
Date:   Sun Nov 17 03:59:57 2013 -0500

    List only adminned groups in anti-hazing select
    
    When selecting groups for the membership update and anti-hazing form, this
    includes only the groups that the user is an admin of, not all groups. With the
    previous commit, this fixes ASA-#181.
Note: See TracTickets for help on using tickets.