[buddypress-trac] [BuddyPress Trac] #8582: Add site membership requests.
buddypress-trac
noreply at wordpress.org
Fri Dec 3 15:32:48 UTC 2021
#8582: Add site membership requests.
-------------------------------------+---------------------
Reporter: dcavins | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 10.0.0
Component: Members | Version: 9.1.1
Severity: normal | Resolution:
Keywords: has-patch needs-testing |
-------------------------------------+---------------------
Comment (by dcavins):
I'm attaching two files:
* 8582.9-just-diff.patch is just the changes made after imath's .8 patch.
It contains the necessary changes for multisite setups, so that the
activation email is interrupted like it is on single site setups. I've
tested all four WPMU registration allowance options, but would love for
other testers to work through each scenario, too.
* 8582.9-combined.patch is a complete patch that you should be able to
apply to a fresh branch and have all of the requests functionality.
Specific questions or thoughts:
I've added a WPMU user meta filter to mirror changes made in the
BP_Signups class. I've put it in `bp-core-filters.php` near other wpmu
filters. Is that the right place for the filter?
The "close" button in the thickbox modal appears outside the modal, to the
right. Is that the correct location? It's a little hard to see over the
dark scrim.
Thanks again for your feedback!
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8582#comment:19>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list