Code review comment for lp:~milner/landscape-client/prevent-client-ui-unicode

Revision history for this message
Mike Milner (milner) wrote :

> It's ugly, but works in the sense that it doesn't crash ;)
>
> But of course the user won't be able to register the machine like this. Should
> we perhaps file a new bug about this to fix it properly? With a dialog box
> perhaps saying that the <field> contained invalid characters.

I do have another branch I started first that adds proper unicode support to the config system but it touches too many core parts just before a release.

I've opened Bug #956612 to remind us that we need a better story here.

« Back to merge proposal