Closed Bug 750803 Opened 10 years ago Closed 10 years ago

[Marketplace Reviewer Tools] Show App manifest inline on review page

Categories

(Marketplace Graveyard :: Reviewer Tools, defect, P4)

defect

Tracking

(Not tracked)

VERIFIED FIXED
2012-06-28

People

(Reporter: eviljeff, Assigned: cvan)

References

Details

Attachments

(1 file)

A lot of the common issues with apps are due to incorrect manifests so it would be good if we could view the manifests contents on the review page.  Ajax? Either on page load or on request with a link/button
Assignee: nobody → cvan
Blocks: 735054
Target Milestone: --- → 6.5.3
Target Milestone: 6.5.3 → 6.5.4
Blocks: 752013
No longer blocks: 735054
Priority: -- → P4
Target Milestone: 2012-05-10 → 2012-05-17
Assignee: cvan → nobody
Component: Admin/Editor Tools → Reviewer Tools
Product: addons.mozilla.org → Marketplace
QA Contact: admin-tools → reviewer-tools
Version: unspecified → 1.0
Target Milestone: 2012-05-17 → ---
This is also part of the spec...

"The manifest URL should be displayed with a link to view the manifest content without having to download and parse it."

Although I'm curious if we should display the raw text of the manifest when it was submitted? Or do an ajax request at the time the reviewer requests to see it, which may or may not match the version they are reviewing?
(In reply to Rob Hudson [:robhudson] from comment #1)
> This is also part of the spec...
> 
> "The manifest URL should be displayed with a link to view the manifest
> content without having to download and parse it."
> 
> Although I'm curious if we should display the raw text of the manifest when
> it was submitted? Or do an ajax request at the time the reviewer requests to
> see it, which may or may not match the version they are reviewing?

In my initial implementation of this I used an <iframe> but because of the Content-Type being served for the webapp Firefox would not render the contents.

It might make sense to do an AJAX request that calls a view of ours where we `urlopen` + read the manifest on the fly. Otherwise, we run into the problem of cross-domain XMLHttpRequests, which are not something we can rely on.

On a related note, we also should be dumping the contents (and not just the hash) of the manifest into the Version object.
It should show the raw text of the manifest at the current time as that's that the user is going to get when they install the app.  The submission manifest is largely irrelevant as no-one will have installed it. (*)

Being able to see the changes in the manifest between submission, approval and at a future point will be useful for re-reviews though.

(*) I'm not sure we should handle the scenario where the manifest has changed between the cron job picking up the change (at midnight, or whatever) and now (the review time).  I've logged bug 760399 for this.
See Also: → 760399
No longer blocks: 752013
blocks bug 766201?
Let's do an async request

    $.get('/en-US/reviewers/fetch_manifest', {'app': '<app_slug>'})

and in the view just

    return requests.get('<manifest>').content

and for good measure we might want

    requests.get('<manifest>').headers
Assignee: nobody → cvan
Blocks: 766201
Target Milestone: --- → 2012-07-05
https://github.com/mozilla/zamboni/commit/2aec5f8
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: 2012-07-05 → 2012-06-28
The ones I checked looked okay at https://marketplace-dev.allizom.org/en-US/reviewers/apps/review/meemoo?num=7&tab=pending

I saw one app throw a 404 but opening the manifest directly also gives an error. So, not too worried about that. See https://marketplace-dev.allizom.org/en-US/reviewers/apps/review/petoo-snowball?num=4&tab=pending

Marking this as verified. If there are issues or new features you'd like, please file a new bug.
Status: RESOLVED → VERIFIED
Attached image post-fix screenshot
You need to log in before you can comment on or make changes to this bug.