Code review comment for lp:~ki7mt/ubuntu-manual-tests/lp1168084-fix

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

I would make the same comments regarding grammar and tags on this merge as on the other merge; https://code.launchpad.net/~ki7mt/ubuntu-manual-tests/server-upgrade-cli/+merge/202185/comments/474664

A couple other thoughts;

1) Don't supply an example username and password; let the user choose
2) I'm confused by the keyboard layout step;

15 + <li>Select layout, "No"</li>
16 + <li>Select country</li>
17 + <li>Select layout</li>

What does this mean?

Overall, good work. Thanks!

review: Needs Fixing

« Back to merge proposal