Closed (fixed)
Project:
User Referral
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Nov 2008 at 15:59 UTC
Updated:
12 Dec 2008 at 00:22 UTC
Jump to comment: Most recent file
Comments
Comment #1
aufumy commentedSets the cookie if you visit a users profile page. This assumes that anonymous viewers can view user profile pages.
Comment #2
eranglr commentedThank you A LOT! (I guess not only from myself).
It work's like a charm, is it implanted in the new release?
Is it possible notify the refer user (the user that own the profile) upon new registration user or/and new flagged user?
Comment #3
aufumy commentedI will have a look tonight. The way to expedite a patch into the project release, is after testing that it works, to change the issue status to "patch (reviewed & tested by the community)".
Comment #4
eranglr commentedok, I'm new to this.
anyway, your patch seems to work well, one scenario (which needs to be mention) could be:
user A visited user B profile, after that went to see user C profile. upon registration, user C will be the one who will get the reffer-credit. maybe there could be a checkbox that will let admin choose the scenario needed (first or last profile viewed).
Also, I had thought about the notification mail, it could be done using userpoints email (http://drupal.org/project/userpoints_contrib).
problems is: first - this module is not yet imported to drupal 6 (part of this contrib ARE ported, but not this feature). secondly, it means that you have to install userpoints + email points modules, just to notify a user upon new reffered/flaged user.
Personally, I like the flexibility of the combination between modules.
if you need my help (by making you coffe - my coding skills suck, just page me :0 ).
btw - how can I get notification email about new comments regarding this/other issues here instead of just going over and over to this thread?
Comment #5
kbahey commentedWe are doing this hodge podge and mixing up all kinds of modules and all kinds of requirements/features.
Let us do this right, without confusing the user with features. Open a new feature for the email notificiation.
I committed aufumy's patch, and it should be in the 6.x-1.x-dev version in about 2 hours or so. So it needs to be tested and verified. If it does not work then reopen this issue.