Closed
Bug 416105
Opened 17 years ago
Closed 17 years ago
Allow you to capture affiliate_id for a download
Categories
(Websites Graveyard :: spreadfirefox.com, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: paul, Assigned: ian)
References
()
Details
Attachments
(1 file)
1.07 KB,
patch
|
morgamic
:
review-
|
Details | Diff | Splinter Review |
Would you please push the following changes to the affiliates module ..
1. When a record is entered into the affiliates table to record an affiliate
referral event we record that at that time no download has been attributed
with that referral event
2. The redirection to the mozilla download page carries the affiliate id for the
affiliate referral
Please also add the field "download" to the affiliates table ..
ALTER TABLE `affiliates` ADD `download` ENUM( 'Y', 'N' ) NOT NULL DEFAULT 'N'
We will then need to work out how we are going to update the download field after a successful download . We will probably want to do this when the "first time" page loads in the new browser.
I am going to give this some further consideration now
I have also commited some minor formatting changes to sfx_affiliates.install
Reporter | ||
Comment 1•17 years ago
|
||
Attachment #301866 -
Flags: review?
Reporter | ||
Comment 2•17 years ago
|
||
I can see two options for determining whether a referral event leads to an actual download but they are both not perfect ..
1. When a user in IE or some other browser clicks on the download button on mozilla
download page we at that point do an XMLHTTPREQUEST to the server to update the affiliates.download = Y against a known affilaite_id
2. Another possibility is when firefox is installed and the start page loads we do an XMLHTTPREQUEST to the server to look for an affiliate referral for the known external IP and update the affiliates.download = Y
I think 1 is by far the better choice as its a fair assumption that a click on the download button will result in an actual download and we would only be doing the XMLHTTPREQUEST for actual affiliate referrals
Any thoughts ?
Updated•17 years ago
|
Attachment #301866 -
Flags: review? → review?(morgamic)
Comment 3•17 years ago
|
||
Since this would require changes to sfx and mozilla.com it probably doesn't belong in the server ops queue quite yet. Moving to the spreadfirefox queue.
Assignee: server-ops → ian-bugzilla
Group: infra
Component: Server Operations: Web Content Push → spreadfirefox.com
Product: mozilla.org → Websites
QA Contact: justin → spreadfirefox-com
Version: other → unspecified
Reporter | ||
Comment 4•17 years ago
|
||
I made a mistake with that patch i passed the user id and not the affiliate id
Would you please delete my previous commit and ill commmit again tomorrow with some additional work that i am currently doing .
Thanks Paul
Comment 5•17 years ago
|
||
Comment on attachment 301866 [details] [diff] [review]
sfx_affiliates.module patch
denied, per comment #4
Attachment #301866 -
Flags: review?(morgamic) → review-
Reporter | ||
Comment 6•17 years ago
|
||
The changes have all been commited with
https://bugzilla.mozilla.org/show_bug.cgi?id=416874
Reporter | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Product: Websites → Websites Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•