1) I just installed this module and noticed that "Do not use Elaborations UI" does not really work. If I check this option, and then try and add a friend (one way relationship), I still get a popup with the form for adding Elaboration. Is there a way that user does not have to explain/add the elaboration when adding a user as a friend.
2) Is there a way to send mail to users when someone becomes their friend (One way relationship which can be reciprocated). I had a look at the User Relationship Mailer options and it seems that mails are only sent when either user has to approve the request etc but there was no option to send a mail when a one way relationship is created between users
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | user_relationship_elaborations.patch | 943 bytes | vegeneric |
| #6 | user_relationship_elaborations.patch | 1.1 KB | vegeneric |
| #1 | user_relationship_elaborations.patch | 1.1 KB | vegeneric |
Comments
Comment #1
vegeneric commentedI ran into your 1) issue as well. I tracked it down in the code, here's a patch that worked for me against 6.x-1.0.
Comment #2
vegeneric commentedComment #4
vegeneric commentedComment #5
berdirComment #6
vegeneric commentedre-uploading with appropriate version tag.
Comment #7
berdir#1: user_relationship_elaborations.patch queued for re-testing.
Comment #8
berdirThe patch has the wrong file path, instead of "a/sites/all/modules/contrib/user_relationships/user_relationship_elaborations/user_relationship_elaborations.module", it just need to be "a/user_relationship_elaborations/user_relationship_elaborations.module". You can either edit the patch file manually or use the --prefix option of git diff.
Comment #9
vegeneric commentedOops, looks like we were both commenting at once. Here's an updated version with corrected path.
Comment #10
berdirComment #11
mrf commentedLooking at issues to include in 6.x-1.1
Comment #12
mrf commentedTested and committed to 6.x-1.x dev.
Thanks for your help!
Has anyone checked if this same issue occurs on 7.x? I'll try and check later if no one else has confirmed.
Comment #13
berdirLooks like we do, commited.