Changes between Initial Version and Version 1 of Ticket #2706, comment 5
- Timestamp:
- 02/09/2017 07:46:31 AM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #2706, comment 5
initial v1 1 1 I 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. 2 2 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. 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. (We would still need one auxiliary plugin though.) 4 4 5 5 Potential 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.