[buddypress-trac] [BuddyPress] #5148: Make notifications a separate component
buddypress-trac
noreply at wordpress.org
Wed Nov 6 18:43:19 UTC 2013
#5148: Make notifications a separate component
----------------------------------+------------------------------
Reporter: johnjamesjacoby | Owner: johnjamesjacoby
Type: defect (bug) | Status: new
Priority: normal | Milestone: 1.9
Component: Notifications | Version:
Severity: normal | Resolution:
Keywords: needs-patch needs-ui |
----------------------------------+------------------------------
Comment (by johnjamesjacoby):
To-do's:
Because there's no full-text of the notification itself, bp-notifications
should be responsible for putting the item_id, secondary_item_id,
component_name, component_action, and date_notiified into real words.
In the current implementation, `notification_callback` is used as a
singlular/cumulative catch-all; `bp_activity_format_notifications()` is a
good example. `bp_core_get_notifications_for_user()` will be used as a
foundation to loop through active notifications, and build the full-text
strings for each line item, without grouping all similar notifications
into 1 to be acknowledged.
We'll also need to change the behavior from `delete` to `mark read` and
make sure `bp_core_get_notifications_for_user()` only shows unread
notifications. This won't be difficult, but will likely touch each
component, since they are individually responsible for knowing when their
notifications are no longer new.
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5148#comment:13>
BuddyPress <http://buddypress.org/>
BuddyPress
More information about the buddypress-trac
mailing list