Bug #3415
closedGroup Details Updates Notifications
0%
Description
I received a message that a group's details had been updated (attached screenshot).
The view link was not correct, This is the link that was given:
http://commons.gc.cuny.edu/groups/stem-teaching-with-technology-colloquium-a-new-catalyst-for-learning/%3C/a%3ESTEM%20Teaching%20with%20Technology%20Colloquium:%20A%20New%20Catalyst%20for%20Learning%3C/a%3E%20were%20updated.%3Cbr%20/%3E%3Cbr%20/%3E%3Ca%20href=/
I'm assuming that by Details it means the Name, Link or Description was updated. Also I suppose the link should have been back to the homepage of the group.
While it's not urgent I can think of a couple of possibly usability modifications related to this:
1. Modify the email with more on what Detail was changed Name, description etc. It would be great if it could read something like: "The group's description has been changed to..."
2. If an email is sent out then also show something in the activity stream with the same text.
Files
Updated by Boone Gorges about 10 years ago
- Status changed from New to Assigned
- Target version set to 1.7
- Estimated time set to 2.00 h
Thanks, Chris. I have corrected the View link https://github.com/cuny-academic-commons/cac/commit/9a52e8789f0c98d5f7a9fd8bbe0c7d906fc8d219
I'm going to move the feature requests to a separate milestone.
Updated by Boone Gorges about 10 years ago
These changes seem appropriate for BP itself, so I'm going to start with BP patches. Then I'll build a plugin for CAC that will serve as a bridge until the changes are integrated into BP.
Ticket for the changes to the email text is here: https://buddypress.trac.wordpress.org/ticket/5879
Updated by Matt Gold about 10 years ago
Awesome. I seem to remember that we are keeping a log somewhere of our upstream contributions to larger free software projects. If so, can you please add this or somehow tag this ticket or the BP ticket so that we can track such things? Many thanks.
Updated by Boone Gorges about 10 years ago
Matt - I will make a note somewhere.
Ticket for the activity item request is here: https://buddypress.trac.wordpress.org/ticket/5880
Updated by Boone Gorges about 10 years ago
- Status changed from Assigned to Testing Required
Added to the Commons 1.7.x branch in https://github.com/cuny-academic-commons/cac/commit/29c944e5e74c029d5f14d718772e9d74ab960e97 and ready for testing on cdev.
A note to myself that before closing this ticket, I should open a new one in the next milestone to check to see whether our custom functionality can be removed yet. It's likely that we'll continue to need at least something for wp-better-emails-for-bp.
Updated by Boone Gorges about 10 years ago
- Status changed from Testing Required to Resolved
Going to mark this one resolved, and we'll handle any bugs during the testing phase.