Re: [Merge] ~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic

2018-11-01 Thread Sebastien Bacher
Review: Approve

K, makes sense, thanks for the work, approving now!
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-initial-setup/+git/gnome-initial-setup/+merge/354195
Your team Ubuntu Desktop is requested to review the proposed merge of 
~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into 
~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic

2018-10-31 Thread Andrea Azzarone
@seb128 This will make sure that the window will be closed but the process will 
keep running until livepatch is installed (or failed to install). The user will 
notice nothing different.
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-initial-setup/+git/gnome-initial-setup/+merge/354195
Your team Ubuntu Desktop is requested to review the proposed merge of 
~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into 
~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic

2018-10-31 Thread Sebastien Bacher
Review: Needs Information

The error handling seems fine to me, could you explain what the "quit" changes 
are about. Is that blocking from closing or from going to the next page until 
you get feedback from the service? What is the typical delay it creates?
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-initial-setup/+git/gnome-initial-setup/+merge/354195
Your team Ubuntu Desktop is requested to review the proposed merge of 
~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into 
~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic

2018-10-29 Thread Matthew Paul Thomas
Review: Approve design

Thanks for the text fixes 
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-initial-setup/+git/gnome-initial-setup/+merge/354195
Your team Ubuntu Desktop is requested to review the proposed merge of 
~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into 
~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into ~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic

2018-09-04 Thread Andrea Azzarone
Few weeks ago I proposed a patch to fix this problem without chaning the 
street. There should be no problem chaning it now.
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-initial-setup/+git/gnome-initial-setup/+merge/354195
Your team Ubuntu Desktop is requested to review the proposed merge of 
~azzar1/ubuntu/+source/gnome-initial-setup:ubuntu/bionic into 
~ubuntu-desktop/ubuntu/+source/gnome-initial-setup:ubuntu/bionic.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop