Project

General

Profile

Actions

Bug #7656

closed

BP notification text is not generated properly on secondary sites when dependent on non-active plugins

Added by Boone Gorges about 7 years ago. Updated about 7 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
BuddyPress (misc)
Target version:
Start date:
2017-02-08
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

We have a number of plugins on the main site that are responsible for creating and formatting BP notifications: bbPress and Social Paper, and perhaps others. On secondary sites, these notifications persist, but the plugin isn't there to format them. As a result, the notification text ends up being 'bbp_new_reply', etc.

See #7624 and https://bbpress.trac.wordpress.org/ticket/2706 for some background.


Related issues

Related to CUNY Academic Commons - Design/UX #7624: BP NotificationsNewPaige Dupont2017-02-04

Actions
Related to CUNY Academic Commons - Bug #7663: Social Paper notifications not formatted correctly on secondary sitesNewBoone Gorges2017-02-11

Actions
Actions #1

Updated by Boone Gorges about 7 years ago

Actions #2

Updated by Boone Gorges about 7 years ago

A first pass at a solution for bbPress is here: https://github.com/cuny-academic-commons/cac/commit/802f2670763bec7229f64e6d43fe989fdeb3a1e6

It's relatively complicated, because of the need to work around CORS restrictions on our subdomain network as well as mapped domains. I've also added a caching layer, so that the async request only takes place when necessary.

Ray, would you mind having a look and letting me know if you think this technique is bonkers? If not, I'll do something similar for Social Paper, which suffers from the same problem.

Actions #3

Updated by Boone Gorges about 7 years ago

  • Status changed from New to Resolved

I'm going to go with this solution for bbPress for now. I'll open a separate ticket for Social Paper so that I can close this one against the 1.10.10 milestone.

Actions #4

Updated by Boone Gorges about 7 years ago

  • Related to Bug #7663: Social Paper notifications not formatted correctly on secondary sites added
Actions

Also available in: Atom PDF