Closed
Bug 523689
Opened 15 years ago
Closed 15 years ago
add download source tracking for install service
Categories
(addons.mozilla.org Graveyard :: Administration, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
5.3
People
(Reporter: fligtar, Assigned: fligtar)
Details
Attachments
(1 file)
1.82 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
The install service should accept a src parameter that it passes to the file download URL so that these add-ons can use source tracking too. So, https://addons.mozilla.org/services/install.php?addon_id=personas&src=external-getpersonas should add the src param to the xpi URL.
Assignee | ||
Comment 1•15 years ago
|
||
And if there is no src= parameter set, let's default to src=installservice and we'll need to add that as an allowed src in the db and add copy to the sources page: From Install Service Downloads passed through the AMO Install Service are counted here.
Summary: src parameter should trickle down install service → add download source tracking for install service
Assignee | ||
Comment 2•15 years ago
|
||
Really want this in 5.3 so doing it myself!
Assignee | ||
Updated•15 years ago
|
Attachment #408750 -
Attachment is patch: true
Attachment #408750 -
Attachment mime type: application/octet-stream → text/plain
Comment 3•15 years ago
|
||
Comment on attachment 408750 [details] [diff] [review] patch, v1 You might want to explain what the AMO Install Service is.
Attachment #408750 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 4•15 years ago
|
||
I was intentionally vague as its existence is not well-known. I may add a little more detail.
Assignee | ||
Comment 6•15 years ago
|
||
I forgot to do SQL for this bug!
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•15 years ago
|
||
eh, I'll just do it in bug 526600
Status: REOPENED → RESOLVED
Closed: 15 years ago → 15 years ago
Resolution: --- → FIXED
Comment 8•15 years ago
|
||
Verified, as this is being tracked/verified in bug 526600.
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Keywords: push-needed
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•