Extension:CodeReview/To-do

Usage notes
Current configuration:
 * Any registered user on the wiki can post comments on bugs
 * Status changes and tagging requires being in the "coder" group. Any bureaucrat or coder can designate another person as a coder. It's viral! ;)

To-do

 * Setup
 * automatic updates done!


 * URL-based UI friendliness
 * grouping the same contributor together, perhaps? (e.g., tstarling vs. timstarling)
 * Special:Code/author should go somewhere useful, like to a list of all authors
 * Not sure what the point of this would be. It already does this is a repo is given. Aaron 20:26, 3 October 2008 (UTC)
 * Perhaps it could say "You haven't specified a repo, please choose one below:" ?
 * You could quite legitimately have a repo called "author". It should be telling you you picked a repo that doesn't exist.


 * Special:Code/MediaWiki/status, /author, /tag etc should give a nice list of available stuff instead of an error :)
 * Done Aaron 15:33, 1 October 2008 (UTC)
 * /author page should be in some sort of logical order (alphabetical?) and perhaps should use two columns (it's quite a long list)
 * Currently done by most-recent commit order (though it is cached) Aaron 20:37, 3 October 2008 (UTC)
 * Could it have a note at the top indicating that that's the order?
 * one of the entries in /author is not a valid title ("Code/MediaWiki/author/"). Was there a commit user named |? Is this just a null value?
 * Shouldn't display anymore. Aaron 20:37, 3 October 2008 (UTC)


 * Tags
 * interface for removing tags
 * Hrm? Aaron 15:33, 1 October 2008 (UTC)
 * now clearer :D
 * Implement advanced tagging system --Catrope 18:51, 2 October 2008 (UTC)
 * Provide tag history (to show who set which tag when) --Catrope 19:01, 2 October 2008 (UTC)
 * Show tags in the main revision list. --Catrope 19:43, 4 October 2008 (UTC)


 * spaces/commas to allow multiple tags
 * Seems to work Aaron 15:33, 1 October 2008 (UTC)
 * it's supposed to auto-lowercase to normalize tags but i'm not sure that's working
 * Seems to work Aaron 15:33, 1 October 2008 (UTC)


 * Auto info on import
 * path-based tags
 * keyword-based tags from commit comment


 * General UI
 * "ok and next" one-click
 * Basic text search
 * Going to Special:RepoAdmin says "The action you have requested is limited to users in the group: Stewards."
 * Do you want a more friendly error message, or a read-only display of the repo information? The latter would prolly be nice.
 * Well, not having it say 'stewards' would be a start. ;-) A read-only display may be nicer, though having people able to see something they can't use might be more effort than it's worth.
 * 'stewards' is correct, since that's the group with the required permission. --brion 21:58, 3 October 2008 (UTC)
 * :-/ Stewards are people at Meta with the ability to change user rights on all WMF wikis. Surely there's another word we could use instead to avoid confusion. The term "local steward" makes me sad, very sad.
 * yeah what i'd like to integrate is running of parser tests, and stick in a count of how test results changed for each rev
 * Special:RecentCodeComments ;-)


 * generally allowing multiple actions at once or else ajax-y faster submits
 * The multiple separate form submissions definitely is not working well; I tend to set multiple field values (status, some tags, a comment) and then try to submit... and the other fields get reset since they weren't submitted. A single giant form (like Bugzilla) will work ok
 * Done Aaron 00:10, 4 October 2008 (UTC)
 * Add PrevNext link at top of page
 * Done Aaron 22:53, 2 October 2008 (UTC)
 * Less commits per page by default and let us change the limit (just like the (20 50 100 500) )
 * Limit select added. 50 is fine. Aaron 22:53, 2 October 2008 (UTC)
 * Add a column with number of comments
 * Done Aaron 23:17, 2 October 2008 (UTC)
 * Enotif for linked local user to committer on comments to their commits


 * Diff handling
 * Add a handler for action=purge to purge the cached diffs; sometimes they fail strangely :(
 * AJAX asynchronous load for not-yet-cached diffs -- this would be nice!