Code review comment for lp:~cyruslien/ubiquity/fix-1716369
Revision history for this message
| Mathieu Trudel-Lapierre (cyphermox) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
I have nothing against guarding this piece of code against failure, but are you sure this is the correct branch to fix the bug you were seeing?
As far as I can tell, the bug open about this is for WPA Enterprise support, and recent rework of ubiquity for WPA should have properly added support for WPA enterprise. Can you please test the behavior without this patch on artful so we know if we should backport things to xenial?