I'm trying out FlaggedRevs to see how far we are from deployment. I'm seeing the following issues right now (local install, default configuration):
* Diffs that are not to the current revision point to the wrong version. (i.e. the review panel at the bottom will tag the current revision, even if neither of the two revs are current). * Edits by users who can review should not need to be reviewed at least at the basic level; is there any way to configure this? It seems pointless to flag edits by trusted users as being in need for vandalism review. * When the last sighted version and the current version are identical, anon users (if so configured) still have to click through to the "current" (identical) version to edit. This seems an unnecessary hoop to jump through.
These seem like core issues to me. In addition, some wishlist items:
* In the original specs we suggested that users can approve unreviewed changes with a collapsible diff + checkbox on the actual edit page when editing an unreviewed version; this still seems like a very simple way to integrate review into normal editing workflow. * Switching the default view for all non-registered visitors would be a very radical thing to do when we haven't figured out yet how scalable the system is. Changes might end up being unreviewed for weeks as a result, rendering articles about current events unusable and making it much harder for new users to discover the site. It seems more sensible to change the default view on a per-page basis for some highly problematic pages which are currently semi-protected, and to gradually increase the number of pages that are in this mode.
I'll try to come up with some more feedback regarding the UI.