Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/ubuntu-weather-app/fix-control-depends-01 into lp:ubuntu-weather-app

Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:196 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/84/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/350/console Click here to trigger a rebuild:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/ubuntu-weather-app/fix-control-depends-01 into lp:ubuntu-weather-app

Andrew Hayzen has proposed merging lp:~ahayzen/ubuntu-weather-app/fix-control-depends-01 into lp:ubuntu-weather-app. Commit message: * Fixes for missing packages in debian/control Requested reviews: Ubuntu Weather Developers (ubuntu-weather-dev) For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/cards-use-gridview into lp:music-app

Review: Approve continuous-integration PASSED: Continuous integration, rev:963 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/85/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/351/console Click here to trigger a rebuild:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/new-protocols-support-02 into lp:ubuntu-filemanager-app

I will fix that, thanks. -- https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/new-protocols-support-02/+merge/280388 Your team Ubuntu File Manager Developers is subscribed to branch lp:ubuntu-filemanager-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/cards-use-gridview into lp:music-app

The proposal to merge lp:~ahayzen/music-app/cards-use-gridview into lp:music-app has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~ahayzen/music-app/cards-use-gridview/+merge/279700 -- Your team Music App Developers is subscribed to branch

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/cards-use-gridview into lp:music-app

The proposal to merge lp:~ahayzen/music-app/cards-use-gridview into lp:music-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ahayzen/music-app/cards-use-gridview/+merge/279700 -- Your team Music App Developers is subscribed to

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/cards-use-gridview into lp:music-app

Review: Approve lgtm! Thanks! :) -- https://code.launchpad.net/~ahayzen/music-app/cards-use-gridview/+merge/279700 Your team Music App Developers is subscribed to branch lp:music-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/20/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1420/console --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app has been updated. Status: Approved => Needs review For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-multiline-support into lp:ubuntu-calculator-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/21/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1421/console --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app

Review: Needs Fixing We would like to press virtual keyboard of calculator, and not keys from On Screen Keyboard (OSK): File "/tmp/adt-run.gw7Oyz/deps/usr/lib/python3/dist-packages/ubuntu_keyboard/emulators/keyboard.py", line 211, in type self.press_key(char) File

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-multiline-support into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-multiline-support into lp:ubuntu-calculator-app has been updated. Status: Approved => Needs review For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app has been updated. Status: Work in progress => Approved For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app has been updated. Status: Needs review => Approved For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-multiline-support into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-multiline-support into lp:ubuntu-calculator-app has been updated. Status: Needs review => Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-bottom-row-fix into lp:ubuntu-calculator-app

Review: Needs Fixing It seems that it is unable to press any key in calculator: 05:18:39.231 INFO _logging:41 - 05:18:39.232 INFO _logging:42 - Starting test

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app

Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:258 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/86/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/352/console Click here to trigger a rebuild:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app

The proposal to merge lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app has been updated. Status: Approved => Needs review For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ubuntu-calculator-dev/ubuntu-calculator-app/ubuntu-calculator-app-fix-autopilot-tests-for-portrait-view into lp:ubuntu-calculator-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/22/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1422/console --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~music-app-dev/music-app/media-hub-bg-playlists-rework into lp:music-app

Review: Needs Fixing This is looking pretty good. I have some inline comments. The only other minor thing is to update the copyright header where appropriate with 2016. Also, I mentioned this before: I think we need to put something in the control file to require the new media-hub and other

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/add_prefix into lp:ubuntu-calendar-app

Looks like Jenkins issue: DEBUG: jenkins job parameter [use_description_for_commit]: not found -- https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/add_prefix/+merge/267317 Your team Ubuntu Calendar Developers is subscribed to branch lp:ubuntu-calendar-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/add_prefix into lp:ubuntu-calendar-app

Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:744 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/80/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/346/console Click here to trigger a rebuild:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1515722 into lp:ubuntu-calendar-app

Looks like Jenkins issue: DEBUG: jenkins job parameter [use_description_for_commit]: not found -- https://code.launchpad.net/~gary-wzl77/ubuntu-calendar-app/fix_1515722/+merge/278759 Your team Ubuntu Calendar Developers is subscribed to branch lp:ubuntu-calendar-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1515722 into lp:ubuntu-calendar-app

Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:744 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/81/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/347/console Click here to trigger a rebuild:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/16/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1416/console --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Review: Approve continuous-integration PASSED: Continuous integration, rev:965 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/83/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/349/console Click here to trigger a rebuild:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Turns out the human is at fault here. The autolanding job(s) were a bit misconfigured on a couple params. -- https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 Your team Music App Developers is subscribed to branch lp:music-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is subscribed to

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/15/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1415/console --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/14/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1414/console --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gary-wzl77/ubuntu-calendar-app/fix_1515722 into lp:ubuntu-calendar-app

Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:744 https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/82/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/348/console Click here to trigger a rebuild:

[Ubuntu-touch-coreapps-reviewers] sboichakov-gmail deactivated by sboichakov-gmail

Hello Ubuntu Core Apps Reviewers, The membership status of Sboichakov Dmitriy (sboichakov-gmail) in the team Ubuntu File Manager Developers (ubuntu-filemanager-dev) was changed by the user from Approved to Deactivated. -- You received this email

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

Review: Needs Fixing continuous-integration FAILED: Autolanding. More details in the following jenkins job: https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-autolanding/13/ Executed test runs: None: https://core-apps-jenkins.ubuntu.com/job/generic-land-mp/1413/console --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app

The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757 -- Your team Music App Developers is