Closed Bug 639982 Opened 13 years ago Closed 13 years ago

Make 'Tested upon' field optional for pending updates

Categories

(addons.mozilla.org Graveyard :: Admin/Editor Tools, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: krupa.mozbugs, Assigned: gkoberger)

References

()

Details

Attachments

(3 files)

Attached image screenshot
steps to reproduce:
1. Load https://addons.allizom.org/z/en-US/editors/review/394267?num=1
2. Click on 'Grant Preliminary review'
3. Add a review comment and click 'Save'


expected behavior:
Review is successful 

observed behavior:
Review is unsuccessful because 'Tested upon:apps' and 'Tested upon:OS' are required fields

We should make those fields optional.
Assignee: nobody → gkoberger
Priority: -- → P2
Target Milestone: 6.0.1 → 6.0.2
Whiteboard: [post-freeze+]
Target Milestone: 6.0.2 → 6.0.1
Whiteboard: [post-freeze+]
Target Milestone: 6.0.1 → 6.0.2
Additionally, we should hide it for the "Request ____" options.
It should be optional for everything, and not even show up if you pick "Request more info" or "Request super review"

https://github.com/jbalogh/zamboni/commit/ac3635481ecc7898a4bf6f8ff408c53009ac976b
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Tested upon info is an optional field for all reviews. Marking this as verified.
Status: RESOLVED → VERIFIED
Attached image post-fix screenshot
Tested upon info doesn't show up super review or request more info
Attached image post
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: