Code review comment for lp:~jtv/launchpad/translationimportqueueentry-info

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

= Bug 519556 =

When reviewing translations uploads, we translations admins often want quick access to the following information on the approval page:
 * The uploader.
 * The file's contents.
 * The project: what's its license status, is it already using Translations?
 * The release series: does it have any templates yet, and which ones?
 * When was the entry last updated? The current UI fails to show this anywhere.

In addition, it may occasionally be convenient to have a copy of the entry's error_output shown. That makes this page a great place to go when analyzing failures.

In this branch I add all of this information. It's at the bottom of the page, since I want it to be unobtrusive. It wouldn't do to force admins to scroll down before they get to the input fields. (Only translations admins and the Ubuntu translation coordinates can access this page).

For Q/A, I intend to visit a whole bunch of pages for translation import queue entries and see that they all make sense.

To test:
{{{
./bin/test -vv -t importqueue/xx-entry
}}}

No lint.

Jeroen

« Back to merge proposal