I really like the fact that you've spent time to get this together, and I like the added feature (and it looks like I had reported the bug initially...); but it was already late in the cycle by the time we got it to a decent point, and I dropped the ball getting it landed sooner and testing it, busy.
Days from release is definitely not the moment to land something like this -- it's a new feature, and one for which there is already a viable workaround (in using nm-applet directly to connect).
*However* I think that we should land this as soon as the Z+1 release is open, this seems like a very good improvement.
Moving to ubuntu-17.05; so that it shows up in my to-do list still for May and I can go back to this bug and do the merge.
@Jeremy,
I really like the fact that you've spent time to get this together, and I like the added feature (and it looks like I had reported the bug initially...); but it was already late in the cycle by the time we got it to a decent point, and I dropped the ball getting it landed sooner and testing it, busy.
Days from release is definitely not the moment to land something like this -- it's a new feature, and one for which there is already a viable workaround (in using nm-applet directly to connect).
*However* I think that we should land this as soon as the Z+1 release is open, this seems like a very good improvement.
Moving to ubuntu-17.05; so that it shows up in my to-do list still for May and I can go back to this bug and do the merge.