Closed (fixed)
Project:
Buddylist
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
5 Aug 2006 at 00:20 UTC
Updated:
1 Nov 2006 at 17:47 UTC
This patch allows email to be sent to users when they are added to, or removed from, someone's buddylist. This email is configurable on the buddylist configurations page and can contain a series of replacement tags such as %adder_name, %addee_name, %site, etc...
Good stuff!
-jeff
| Comment | File | Size | Author |
|---|---|---|---|
| buddylist-mail.patch | 6.7 KB | jjeff |
Comments
Comment #1
ajwwong commentedThis seems to work quite well...
Excellent!! Thanks so much...
The one thing that I couldn't get to work was filling in the "alternate text" in the form. I'm not sure if I applied the patch right, but for some reason, even though all the variables were appropriately set, e.g., buddylist_add_message and buddylist_remove_message [in the final buddy_mail_user function], I still couldn't get those messages to substitute in instead of the hard-coded defaults.
Thanks, though for a big step forward for buddylist!
Comment #2
robertdouglass commentedThanks for testing. Please confirm that you tested against the latest 4.7.0 branch and not CVS. Any chance that you could look into the issue with the replacement variables and reroll?
Comment #3
ajwwong commentedHey Robert,
Just fyi.... I'm testing on a heavily modded-out version of my own "friends by mutual consent / invite-only" style buddylist module... and therefore I did this test as hand applied (and modified for my own needs patch) .... so my test above is definitely not a "controlled testing environment" :-)
But maybe someone who's working with a fresh copy of buddylist can run with it... 'cause this is definitely a *great* feature addition for buddylist.
I'm kinda swimming in my own stuff just trying to keep up with the some of the new traffic on my site and make sure it's up and running. But I'll definitely try to look into this more when I can.
Take care,
Albert
Comment #4
thierry_gd commentedWorks fine for me
Can't wait to see this feature added to HEAD
Comment #5
robertdouglass commentedthanks for the review. Taking a look at it today.
Comment #6
robertdouglass commentedcommitted to 4.7
Comment #7
robertdouglass commentedComment #8
(not verified) commented