Closed
Bug 786867
Opened 12 years ago
Closed 12 years ago
Allow editors to download (and install) packaged apps from the Reviewer details page
Categories
(Marketplace Graveyard :: Reviewer Tools, defect, P5)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: krupa.mozbugs, Unassigned)
References
Details
(Whiteboard: [see comment 7])
Install of packaged apps is not supported on desktop. However, we need a way for editors to download and look at the contents of the packaged app.
So, we should switch "install" button to "download" on Reviewer Details page and allow editors to download packaged apps.
Updated•12 years ago
|
Blocks: market-packaged-apps
Summary: Allow editors to download packaged apps from the Reviewer details page → Allow editors to download (and install) packaged apps from the Reviewer details page
Comment 1•12 years ago
|
||
Andrew: what do you want to do here?
Comment 2•12 years ago
|
||
possibly crazy idea:
Could the install <button> on the review page be change to be <a> that looks like a button and installs the same as before when left clicked but acts like a link otherwise (so right click; save as, or middle click would work for a download)?
Otherwise two buttons is acceptable.
For viewing via Mkt I was expecting a 'View Contents' link like addons have. Bug 784200 mentions the Compare diff viewer and its essentially the same thing.
Updated•12 years ago
|
Priority: -- → P2
Comment 3•12 years ago
|
||
(In reply to Andrew Williamson [:eviljeff] from comment #2)
> possibly crazy idea:
> Could the install <button> on the review page be change to be <a> that looks
> like a button and installs the same as before when left clicked but acts
> like a link otherwise (so right click; save as, or middle click would work
> for a download)?
I'm confused
Comment 4•12 years ago
|
||
Now that we have a "mini manifest" some of this doesn't make sense...
I've made the manifest "View" link show the mini manifest. That's now what gets sent to the install API. In that JSON is a path to the zip file you can click on to download the zip file.
So via those tools you have both install and download package ability.
Comment 5•12 years ago
|
||
(In reply to Chris Van Wiemeersch [:cvan] from comment #3)
> (In reply to Andrew Williamson [:eviljeff] from comment #2)
> > possibly crazy idea:
> > Could the install <button> on the review page be change to be <a> that looks
> > like a button and installs the same as before when left clicked but acts
> > like a link otherwise (so right click; save as, or middle click would work
> > for a download)?
>
> I'm confused
the install button would have been a link to the download package styled as a button but with a left-click onclick handler that did the installing. But as Rob mentions, I think the mini-manifest makes this partially irrelevant (it unlikely we'd want to download the mini-manifest) so the link in the 'View' link will do.
Comment 6•12 years ago
|
||
Does that mean this bug is done? It's a bit roundabout (having to click the manifest, then the URL inside it) and the package listed in the mini manifest is (will be) the signed package for reviewers. Would it be better to have a download button which links directly to the uploaded package?
Comment 7•12 years ago
|
||
The current implementation (download via the mini-manifest) is acceptable to stop this blocking.
An extra download button that links directly to the uploaded package would be great though if its not a lot of additional work!
Comment 8•12 years ago
|
||
Sure thing.
Updated•12 years ago
|
Priority: P2 → P5
Whiteboard: [see comment 7]
Comment 9•12 years ago
|
||
The title of this bug is confusing. Based on comment 7 and that we have a file viewer implemented now, I'm going to wontfix.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•