Code review comment for lp:~happyaron/ubiquity/lp-1465530

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks fine, but it's also broken in localechooser, and this omits fixing the case for the LANGLIST= values.

Rejecting; but I'll fix this in localechooser and in the patch and credit you for the changes.

review: Needs Fixing

« Back to merge proposal