316930 - Test accounts for addons which require a login

Status: RESOLVED FIXED

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

Description

[Cameron]

Addon authors should be able to provide test accounts for extensions which require a login (eg. website integation.) This will make reviewing such extensions easier and speed up the process.

Needs new field in the database. Probably a "developer comments to reviewers" so it can include test account details, changelogs, etc. 

Should be avaliable in 1.0, later we can put it in 2.0 aswell.

Comment 1

[Mike Shaver (:shaver emeritus)]

Punting back to the triage pool; this would help a lot.

Comment 2

[Justin Scott [:fligtar]]

Attachment: Review Notes Patch

Patches developers/approval.php and developers/additem.php to support a text field for Notes to Reviewers.

Comment 3

[Justin Scott [:fligtar]]

Attachment: SQL query

This SQL query will need to be run on the version table to add a text field for the ReviewNotes.

Comment 4

[Cameron]

*** Bug 291908 has been marked as a duplicate of this bug. ***

Comment 5

[Justin Scott [:fligtar]]

Attachment: Review Notes Patch (updated)

This patches developers/approval.php, developers/additem.php, and developers/listmanager.php. It also adds a link to Item History for bug 347690 so I don't have to repatch that as well. The SQL attached will still need to be run.

Comment 6

[Michael Morgan [:morgamic]]

Comment on attachment 232797 [details] [diff] [review]
Review Notes Patch (updated)

Awesome -- looks good.  Checked in.

Comment 7

[Michael Morgan [:morgamic]]

Comment on attachment 232558 [details]
SQL query

Looks good -- will add this to the deployment IT request.