[buddypress-trac] [BuddyPress Trac] #8675: invites.php should query the displaued user not the logged in users invites

buddypress-trac noreply at wordpress.org
Sat Apr 9 06:45:58 UTC 2022


#8675: invites.php should query the displaued user not the logged in users invites
----------------------------------------+---------------------
 Reporter:  shawfactor                  |       Owner:  (none)
     Type:  defect (bug)                |      Status:  new
 Priority:  normal                      |   Milestone:  11.0.0
Component:  Groups                      |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  dev-feedback needs-refresh  |
----------------------------------------+---------------------
Changes (by imath):

 * keywords:  has-patch dev-feedback => dev-feedback needs-refresh


Comment:

 As explained in a review to @oztaser 's PR:
 https://github.com/buddypress/buddypress/pull/15#pullrequestreview-937082276

 I understand why we were doing so, if an admin accepts the invite it
 generates some errors on both template packs.

 We'd first need to work on:
 - `groups_action_join_group()`
 - `bp_nouveau_ajax_joinleave_group()`

 In these functions we are using `bp_logged_in_user_id()`...

 More globally I'm wondering, if we were to go this way if we should also
 generate an email to inform the user the Admin accepted the Group invites
 on behalf of the user. Deciding for a user is always a concern for me.

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


More information about the buddypress-trac mailing list