672872 - Detailed validation report is missing warnings

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect)

Description

[krupa raj[:krupa]]

Attachment: test add-on

steps to reproduce:
1. Load
2. Upload the attached add-on with app=Firefox and version=6.*
3. Notice that validation passes with 10 warnings
4. Click on the link to view the detailed report

actual behavior:
detailed report doesn't list any warnings.

Comment 1

[krupa raj[:krupa]]

Attachment: screenshot

Comment 2

[Kumar McMillan [:kumar]]

this is a UI fix. It doesn't actually have 10 warnings.

Comment 3

[Kumar McMillan [:kumar]]

Fixed https://github.com/jbalogh/zamboni/commit/e41fe13

Comment 4

[Kumar McMillan [:kumar]]

Whoops, the UI also looks at error type. Fixed https://github.com/jbalogh/zamboni/commit/ff1eca1

Comment 5

[krupa raj[:krupa]]

verified at https://addons.allizom.org/z/en-us/developers/addon/check-compatibility

Comment 6

[krupa raj[:krupa]]

Attachment: post-fix screenshot