#7175 closed enhancement (fixed)
Display and filter by group type on Dashboard > Groups
Reported by: | mercime | Owned by: | dcavins |
---|---|---|---|
Milestone: | 2.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Groups | Keywords: | |
Cc: |
Description
This adds a corresponding Group Type implementation of the prospective Member Type display and filter in wp-admin per #6060. To rephrase what Boone posted in that ticket:
Now that we support group types in #6784 and have an admin UI in the Edit Group
screen to view/set the group type on an individual basis, we should consider the following enhancement to Dashboard > Groups
:
- Group Type (sortable) column
- Ability to filter by Group Type. Would be great if the filter method used (either ala Role Links or dropdown) would be similar to what would be implemented for the Member Type filter.
Attachments (1)
Change History (9)
This ticket was mentioned in Slack in #buddypress by dcavins. View the logs.
8 years ago
#4
follow-up:
↓ 5
@
8 years ago
- Milestone changed from Future Release to 2.7
Since you asked:
- bp_current_user_can vs current_user_can :)
- strict comparision operators please :)
- with in_array, please set the third parameter to true (good habit to get into) :)
- In column_content_group_type(), escape the
singular_name
label within that link. :) $columns[ 'bp_group_type' ]
should be$columns['bp_group_type']
. The spacing is only needed if you're using an variable to access an array element, e.g.$my_array[ $my_var ]
. :)- In add_group_type_bulk_change_select(), please escape the all the things. :)
:)
#5
in reply to:
↑ 4
@
8 years ago
Replying to DJPaul:
Since you asked:
- bp_current_user_can vs current_user_can :)
- strict comparision operators please :)
- with in_array, please set the third parameter to true (good habit to get into) :)
- In column_content_group_type(), escape the
singular_name
label within that link. :)$columns[ 'bp_group_type' ]
should be$columns['bp_group_type']
. The spacing is only needed if you're using an variable to access an array element, e.g.$my_array[ $my_var ]
. :)- In add_group_type_bulk_change_select(), please escape the all the things. :)
:)
Thanks for taking the time to read over the patch! That's very helpful.
Related:
#7179 UI to Add New Group Type in wp-admin
#7180 UI for All Group Types screen in wp-admin