Feature #3418
closedRevisions/Additions to email notifications about group membership requests
0%
Description
Just received a bunch of notifications for people to join a public group. A few suggested changes to the notification email:
1. If a note is included in the request, it (or an excerpt, if it is long) should show up in the email notification
2. there should be a line space between "X wants to join Y Group" and "View Notifications."
Files
Updated by Boone Gorges over 10 years ago
- Target version set to 1.7
These seem like reasonable requests that should probably be accomodated at the level of BuddyPress. I'm going to put this ticket into the 1.7 milestone so I remember to move the requests over to the BP tracker, though it's unlikely that they'll appear in BuddyPress in time for CAC 1.7.
2. there should be a line space between "X wants to join Y Group" and "View Notifications."
Can you please share a screenshot of this? I don't see where the text View Notifications appears in this email, though I may be looking in the wrong place.
Updated by Matt Gold over 10 years ago
Thanks. Please see a screenshot attached. And, yes, you're right that "View Notifications" does not appear . . . but all of the links in the message are a bit jammed together.
Updated by Boone Gorges over 10 years ago
Thanks for the screenshot. What you're seeing there is a bug that I've fixed in https://github.com/cuny-academic-commons/cac/commit/d461c86bb927abc88eb848c07862d37488639d92. Leaving the ticket in the 1.7 milestone for the first suggestion.
Updated by Boone Gorges over 10 years ago
BuddyPress ticket: https://buddypress.trac.wordpress.org/ticket/5885
Updated by Matt Gold over 10 years ago
Awesome. I know you are keeping track of upstream contributions, but I'm just going to add #upstream here to make such tickets easily searchable here.
Updated by Boone Gorges over 10 years ago
- Status changed from Assigned to Testing Required
Added to CAC in https://github.com/cuny-academic-commons/cac/commit/6db6e30566a3b483f6e5f28d9483b583c8872bf3 and ready to test on cdev
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.