[1.3.2] Add-on Validator page for non-hosted add-ons

VERIFIED FIXED in 5.1

Status

defect
P1
normal
VERIFIED FIXED
10 years ago
3 years ago

People

(Reporter: fligtar, Assigned: rjwalsh)

Tracking

unspecified

Details

(URL)

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

10 years ago
One of the tools available in the developer hub is the add-on validator even if the add-on is not hosted on AMO.

I will attach a mockup here and more details shortly.
(Reporter)

Updated

10 years ago
Assignee: nobody → fligtar
(Reporter)

Updated

10 years ago
Whiteboard: ->rj
(Reporter)

Comment 1

10 years ago
Posted image mockup
* The Validator page of the developer hub should have a brief introductory paragraph, followed by 2 methods of running the validation tests.
* If the user is not logged-in, instead of the area to select the add-on/file, there should be text that says "You must _log_in_ to validate an add-on". We don't want random people uploading crap.
* If the user is logged-in and has add-ons hosted, they should see the dropdown shown in the mockup where selecting one of their add-ons will take them to the versions & files page for that add-on so they can run tests.
* All logged-in users should see the second part, where an add-on can be uploaded to be tested. The validator should run the tests for this add-on and immediately the delete the file.
(Reporter)

Comment 2

10 years ago
RJ, I know it's your last day tomorrow. If at all possible, could you take a look at implementing a new page in the dev tools with the upload box that takes any add-on regardless of hosted status? We can work on implementing the rest of the UI for this page, but it would definitely help if you could do the upload and validation part.
Assignee: fligtar → rjbuild1088
Whiteboard: ->rj
(Assignee)

Comment 3

10 years ago
Justin - This should be pretty easy to implement now that validation on upload is working.  Even if this isn't done by tomorrow, school doesn't really start for a few weeks, so I can help out here and there where needed.
(Assignee)

Comment 4

10 years ago
Posted patch v1 (obsolete) — Splinter Review
First stab at this - look for the new 'Validate add-on' link in the developer sidebar
Attachment #397332 - Flags: review?(clouserw)
Attachment #397332 - Flags: review?(clouserw) → review-
Comment on attachment 397332 [details] [diff] [review]
v1

The design is fine, but you're duplicating big chunks of code from uploader.thtml.  If that code is going to exist in more than 1 place it should be under /views/elements
(Assignee)

Comment 6

10 years ago
Attachment #397332 - Attachment is obsolete: true
Attachment #397345 - Flags: review?(clouserw)
Comment on attachment 397345 [details] [diff] [review]
v2, refactored

You're probably going to get some conflicts with the new dev nav bar
Attachment #397345 - Flags: review?(clouserw) → review+
(Assignee)

Comment 8

10 years ago
Fixed in r51313
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
When on https://preview.addons.mozilla.org/en-US/developers/addon/validate , logged in as admin, I do select an add-on hosted by AMO, and it takes me to https://developers/addon/validate/$addonid

Anyway, it works if a one uploads an add-on.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 10

10 years ago
Removing an unnecessary /
Attachment #402987 - Flags: review?(clouserw)
Attachment #402987 - Flags: review?(clouserw) → review+
(Assignee)

Comment 11

10 years ago
Fixed in r52388
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED
I get "No test results found." for any test group now. When i upload addon for validation, it looks like it actually running some tests, and when I test uploaded addon, it just gives me that notification.

Tests for uploaded addon are still working fine.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 13

10 years ago
(In reply to comment #12)
> I get "No test results found." for any test group now. 

This is because it just takes you to the validation page, but does not actually run any tests.  We could add that, but it changes the scope of the bug.
But what's the purpose of another method than(other than upload)? As far as I can see, Justin wanted both methods working.
(Assignee)

Comment 15

10 years ago
Milos, have you clicked on the validate button?  It's a bit hidden now since it's not styled, but that runs the validation tests on that page.
Works fine when I try it using both the drop down and uploading a new file.
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.