Feature #3941
closedGroup forums -- Edited comment notifications
0%
Description
A member of one of my groups posted a comment on a forum in response to a post. He then saw an error in his comment, so he clicked "edit," fixed the error, and clicked save.
This produced two email notifications in a short amount of time -- at least to my account (not sure whether non-admins received them, too).
Both notifications came with the same wording -- [user] replied to the topic [topic name] in the forum [forum topic name]
My suggestions:
If an edit of a forum post triggers a notification, it should have a different notification message, e.g.c "[user] edited a reply to [forum topic name]
or
There should not be a second notification if, say, it is posted within a given amount of time from the original post
FYI, here is the post in question (SF's comment)
Updated by Boone Gorges almost 10 years ago
- Assignee changed from Boone Gorges to Daniel Jones
- Target version set to 1.8
Dan, will you look at this? I would suggest getting a checkout of bbPress (git://bbpress.git.wordpress.org) and writing a version of Matt's first suggestion (User edited a reply...) as a patch for bbPress. Once you've got a patch written, I can walk you through submitting it to bbPress.
Updated by Raymond Hoh almost 10 years ago
I added a patch on bbPress Trac awhile ago about this bug:
https://bbpress.trac.wordpress.org/ticket/2690
It should be as easy as applying my patch. Do take note of what I mention here.
If we don't want to patch bbPress directly, we can go with a version of a fix I placed in CBOX.
Updated by Boone Gorges almost 10 years ago
- Target version changed from 1.8 to 1.7.17
Oof, I didn't realize this was the same issue. Let's just go with the bbPress hotfix. Dan, can you please take care of that in the 1.7.x branch?
Updated by Daniel Jones almost 10 years ago
Sorry running into an issue where I can't find the equivalent to CBOX's includes/frontend-bbpress.php in the CAC files. Is there a place where we keep these kinds of modifications to bbpress in CAC already? Should I apply the patch that Ray linked to to bbpress directly instead?
Updated by Boone Gorges almost 10 years ago
We don't have a frontend-bbpress.php equivalent. Please just apply the bbPress hotfix.
Updated by Daniel Jones almost 10 years ago
Updated by Boone Gorges over 9 years ago
- Target version changed from 1.7.17 to 1.7.18
Updated by Boone Gorges over 9 years ago
- Target version changed from 1.7.18 to 1.7.19
Updated by Boone Gorges over 9 years ago
- Target version changed from 1.7.19 to 1.7.20
Updated by Boone Gorges over 9 years ago
- Target version changed from 1.7.20 to 1.7.21
Updated by Boone Gorges over 9 years ago
- Target version changed from 1.7.21 to 1.8.1
Updated by Daniel Jones over 9 years ago
Just wanted to check on this - did my application of the hotfix work?
Updated by Boone Gorges over 9 years ago
- Status changed from Assigned to Resolved
I haven't heard anything, so I'm assuming yes. Let's mark this resolved.