Skip to:
Content

bbPress.org

Changes between Initial Version and Version 1 of Ticket #2706, comment 5


Ignore:
Timestamp:
02/09/2017 07:46:31 AM (3 years ago)
Author:
r-a-y
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2706, comment 5

    initial v1  
    11I definitely applaud the idea of using the WP REST API here, but how about saving the rendered notification URL string directly into BP notification meta at the time of adding the notification?  Then, we reference the notification meta entry when we need to call a single `bbp_new_reply` notification on a sub-site that doesn't have bbPress activated.
    22
    3 This side-steps all the complexity of needing two plugins and a JS file, but it also solves the site ID issue and needing to use either `switch_to_blog()` or the WP REST API here.
     3This side-steps all the complexity of needing two plugins and a JS file, but it also solves the site ID issue and needing to use either `switch_to_blog()` or the WP REST API here.  (We would still need one auxiliary plugin though.)
    44
    55Potential problems with my idea is stale data such as if the topic title is edited or if the person who posted a reply has changed their display name.  The other issue is perhaps, the over-reliance on notification meta.