[buddypress-trac] [BuddyPress Trac] #5336: Add "manage invitations" pane to group admin screen

buddypress-trac noreply at wordpress.org
Sun Feb 23 16:49:38 UTC 2014

#5336: Add "manage invitations" pane to group admin screen
 Reporter:  dcavins                      |       Owner:
     Type:  enhancement                  |      Status:  new
 Priority:  normal                       |   Milestone:  2.0
Component:  Administration               |     Version:  1.9.1
 Severity:  normal                       |  Resolution:
 Keywords:  has-patch reporter-feedback  |

Comment (by dcavins):

 Here's a first pass at adding pagination to `bp_group_has_invites`.

 Full disclosure: I changed some default behaviors. As it is now, when a
 user visits a group's "send invites" pane, he is only shown invitations
 that he has created (even if he is a group or site admin). This causes
 problems for group members who can't figure out why they can't invite
 someone to a group (because someone else already did, but they can't see
 that). I changed the behavior so that all outstanding group invitations
 are show to all members, but only the inviter or group or site admins can
 "remove" an invite. I hope this is a move that will be OK with everyone.

 Also, clicking "send invitations" will try to send all of the unsent
 invitations for the group. I've gotten some complaints from my users that
 they don't know to click the "send invites" button, so I'm wondering about
 that, too. One thought is to remove it, and add a "confirm invite" button
 in the action div next to "remove" for each invite. That doesn't handle
 bulk actions very well, though. Another approach is to add a dialog that
 reminds the user to click the button if they're leaving the page by
 navigating away. I can add either, if you think they're a good idea.

 Finally, since I reordered the arguments get_invite
 `BP_Groups_Group::get_invites` I left `groups_get_invites_for_group()` in
 place and added a new function, `groups_get_all_invites_for_group`,
 because I didn't want to break uses of the old function.

 Please offer any comments, and I'll rework the code as necessary.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5336#comment:7>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac

More information about the buddypress-trac mailing list