[buddypress-trac] [BuddyPress Trac] #8139: Network Invitations and Membership Requests

[buddypress-trac]

#8139: Network Invitations and Membership Requests
--------------------------+----------------------
 Reporter:  dcavins       |       Owner:  dcavins
     Type:  enhancement   |      Status:  closed
 Priority:  normal        |   Milestone:  8.0.0
Component:  Registration  |     Version:  5.0.0
 Severity:  normal        |  Resolution:  fixed
 Keywords:  needs-patch   |
--------------------------+----------------------

Comment (by dcavins):

 Replying to [comment:77 imath]:
 > I've also edited the `'bp_members_invitations_view_send_screen'`
 capability check as when I'm logged in as a regular user I wasn't able to
 view the restricted message you used into your screenshot. Maybe you
 should give an extra check about this one 😉.
 >
 Hi imath, you won't get that custom message unless a plugin is providing
 it. You can see my proof-of-concept invite limiter code here
 https://github.com/dcavins/bp-invitation-limiter

 Regarding your other questions:
 • Sure I can put all the actions-hooked items into the actions file. I
 sort of don't like that filing system (it's a little like putting all of
 your spreadsheets in one directory on your desktop), because I would
 prefer functionally related items to be stored together--like all the
 invitations code together rather than filed by whether it's a filter or
 action-hooked function. But I don't have strong feelings about it.

 Thanks again for your review!

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8139#comment:81>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac