[buddypress-trac] [BuddyPress Trac] #8734: Introduce simple "Private Site" toggle
buddypress-trac
noreply at wordpress.org
Tue Oct 18 12:56:55 UTC 2022
#8734: Introduce simple "Private Site" toggle
-----------------------------+-----------------------
Reporter: dcavins | Owner: (none)
Type: feature request | Status: assigned
Priority: normal | Milestone: 11.0.0
Component: Core | Version: 10.4.0
Severity: normal | Resolution:
Keywords: has-patch |
-----------------------------+-----------------------
Comment (by sbrajesh):
Hi @imath
Thank you for the work on it. It is going in the right direction. I have a
few suggestions and will love to hear your feedback.
1. As @johnjamesjacoby points, the context parameter(non id) seems a bit
out of the place there. We should avoid it.
2. It would be nice to have a function similar to the function
bp_user_has_access() which provides a simple filter, context and uses
bp_read cap.
3. Also, I am not sure if introducing additional complexity is required.
We can let the original parsing work as it did earlier. Then, we simply
hook to 'bp_replace_the_content' and if the community is not visible, we
just discard any other hooked content and load the community gate
template.
Just some thoughts to keep it simple. It is alright if you decide to go
with the current implementation.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8734#comment:29>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list