[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