[Ubuntu-touch-coreapps-reviewers] [Bug 1552100] [NEW] Move to .snap

2016-03-01 Thread Emanuele Antonio Faraone
Public bug reported:

Move all packages to .snap ( snappy)

** Affects: canonical-devices-system-image
 Importance: Undecided
 Status: New

** Affects: music-app
 Importance: Undecided
 Status: New

** Affects: reminders-app
 Importance: Undecided
 Status: New

** Affects: telegram-app
 Importance: Undecided
 Status: New

** Affects: ubuntu-calendar-app
 Importance: Undecided
 Status: New

** Affects: ubuntu-docviewer-app
 Importance: Undecided
 Status: New

** Affects: ubuntu-filemanager-app
 Importance: Undecided
 Status: New

** Affects: ubuntu-terminal-app
 Importance: Undecided
 Status: New

** Affects: ubuntu-weather-app
 Importance: Undecided
 Status: New

** Also affects: ubuntu-calendar-app
   Importance: Undecided
   Status: New

** Also affects: music-app
   Importance: Undecided
   Status: New

** Also affects: ubuntu-terminal-app
   Importance: Undecided
   Status: New

** Also affects: ubuntu-filemanager-app
   Importance: Undecided
   Status: New

** Also affects: reminders-app
   Importance: Undecided
   Status: New

** Also affects: ubuntu-docviewer-app
   Importance: Undecided
   Status: New

** Also affects: ubuntu-weather-app
   Importance: Undecided
   Status: New

** Also affects: telegram-app
   Importance: Undecided
   Status: New

-- 
You received this bug notification because you are a member of Ubuntu
File Manager Developers, which is subscribed to Ubuntu File Manager App.
Matching subscriptions: File Manager App Bugmail
https://bugs.launchpad.net/bugs/1552100

Title:
  Move to .snap

Status in Canonical System Image:
  New
Status in Ubuntu Music App:
  New
Status in Ubuntu Notes app:
  New
Status in Telegram app:
  New
Status in Ubuntu Calendar App:
  New
Status in Ubuntu Document Viewer App:
  New
Status in Ubuntu File Manager App:
  New
Status in Ubuntu Terminal App:
  New
Status in Ubuntu Weather App:
  New

Bug description:
  Move all packages to .snap ( snappy)

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1552100/+subscriptions

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:766
https://core-apps-jenkins.ubuntu.com/job/calendar-app-ci/768/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/664/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/calendar-app-ci/768/rebuild

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Jenkins Bot
The proposal to merge lp:~mpt/reminders-app/1551780-setup into lp:reminders-app 
has been updated.

Status: Approved => Needs review

For more details, see:
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
-- 
Your team Ubuntu Notes app developers is subscribed to branch lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Autolanding.
No commit message was specified in the merge proposal. Hit 'Add commit message' 
on the merge proposal web page or follow the link below. You can approve the 
merge proposal yourself to rerun.
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658/+edit-commit-message

-- 
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
Your team Ubuntu Notes app developers is subscribed to branch lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Michael Zanetti
The proposal to merge lp:~mpt/reminders-app/1551780-setup into lp:reminders-app 
has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
-- 
Your team Ubuntu Notes app developers is subscribed to branch lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Michael Zanetti
Review: Approve

thanks!
-- 
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
Your team Ubuntu Notes app developers is subscribed to branch lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~renatofilho/ubuntu-calendar-app/bottom-edge into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:763
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1106/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/663/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1106/rebuild

-- 
https://code.launchpad.net/~renatofilho/ubuntu-calendar-app/bottom-edge/+merge/284622
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread Jenkins Bot
Review: Approve continuous-integration


-- 
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675
Your team Ubuntu Calculator Developers is subscribed to branch 
lp:ubuntu-calculator-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread noreply
The proposal to merge lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 
into lp:ubuntu-calculator-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675
-- 
Your team Ubuntu Calculator Developers is subscribed to branch 
lp:ubuntu-calculator-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread Jenkins Bot
Review: Approve continuous-integration

PASSED: Continuous integration, rev:279
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1105/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/662/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1105/rebuild

-- 
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675
Your team Ubuntu Calculator Developers is subscribed to branch 
lp:ubuntu-calculator-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread Alan Pope 
Review: Approve

Built deb and click successfully here.
-- 
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675
Your team Ubuntu Calculator Developers is subscribed to branch 
lp:ubuntu-calculator-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread Alan Pope 
The proposal to merge lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 
into lp:ubuntu-calculator-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675
-- 
Your team Ubuntu Calculator Developers is subscribed to branch 
lp:ubuntu-calculator-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into lp:ubuntu-calculator-app

2016-03-01 Thread David Planella
David Planella has proposed merging 
lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into 
lp:ubuntu-calculator-app.

Commit message:
Fix .deb package failure to build

Requested reviews:
  Ubuntu Calculator Developers (ubuntu-calculator-dev)
Related bugs:
  Bug #1551848 in Ubuntu Calculator App: "Debian package fails to build"
  https://bugs.launchpad.net/ubuntu-calculator-app/+bug/1551848

For more details, see:
https://code.launchpad.net/~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848/+merge/287675

This fixes the failure of the .deb package to build.

To test it:

- Simply build the .deb package. Before the fix it fails to build.
- The change should not affect the click package in any way, but it wouldn't 
hurt to build it to be extra certain.
-- 
Your team Ubuntu Calculator Developers is requested to review the proposed 
merge of lp:~dpm/ubuntu-calculator-app/fix-deb-build-bug-1551848 into 
lp:ubuntu-calculator-app.
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2016-01-13 23:37:57 +
+++ CMakeLists.txt	2016-03-01 17:00:58 +
@@ -85,7 +85,7 @@
 DESTINATION ${CMAKE_INSTALL_BINDIR})
 endif(CLICK_MODE)
 
-install(FILES app/${ICON} DESTINATION ${UBUNTU-CALCULATOR-APP_DIR}/graphics)
+install(FILES app/${ICON_FILE} DESTINATION ${UBUNTU-CALCULATOR-APP_DIR}/graphics)
 
 file(GLOB_RECURSE I18N_SRC_FILES
  RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/po

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Matthew Paul Thomas
The proposal to merge lp:~mpt/reminders-app/1551780-setup into lp:reminders-app 
has been updated.

Description changed to:

Shortens and copyedits text in the Evernote sync dialog, fixing bug 1551780.

Also changes the dialog buttons:
* “No” to “Not Now”, since you can change your mind later
* red to grey, since a single dialog shouldn't have both green and red buttons
* “Yes” to “Set Up…”, since you have to do something more to set up the account.

For more details, see:
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
-- 
Your team Ubuntu Notes app developers is requested to review the proposed merge 
of lp:~mpt/reminders-app/1551780-setup into lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Matthew Paul Thomas
The proposal to merge lp:~mpt/reminders-app/1551780-setup into lp:reminders-app 
has been updated.

Description changed to:

Shortens and copyedits text in the Evernote sync dialog, fixing bug 1551780.

Also changes "No" to "Not Now" (since you can change your mind later) and from 
red to grey (since a single dialog shouldn't have both green and red buttons).

For more details, see:
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658
-- 
Your team Ubuntu Notes app developers is requested to review the proposed merge 
of lp:~mpt/reminders-app/1551780-setup into lp:reminders-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mpt/reminders-app/1551780-setup into lp:reminders-app

2016-03-01 Thread Matthew Paul Thomas
Matthew Paul Thomas has proposed merging lp:~mpt/reminders-app/1551780-setup 
into lp:reminders-app.

Requested reviews:
  Ubuntu Notes app developers (notes-app-dev)
Related bugs:
  Bug #1551780 in reminders-app (Ubuntu): "Intro uses “setup” as a verb"
  https://bugs.launchpad.net/ubuntu/+source/reminders-app/+bug/1551780

For more details, see:
https://code.launchpad.net/~mpt/reminders-app/1551780-setup/+merge/287658

Shortens and copyedits text in the Evernote sync dialog, fixing bug 1551780.
-- 
Your team Ubuntu Notes app developers is requested to review the proposed merge 
of lp:~mpt/reminders-app/1551780-setup into lp:reminders-app.
=== modified file 'src/app/qml/Reminders.qml'
--- src/app/qml/Reminders.qml	2015-11-20 11:11:49 +
+++ src/app/qml/Reminders.qml	2016-03-01 16:08:15 +
@@ -227,7 +227,7 @@
 switch (accounts.count) {
 case 0:
 PopupUtils.open(noAccountDialog, root);
-print("No account available! Please setup an account in the system settings");
+print("No account available. Please set up an account in System Settings.");
 break;
 case 1:
 print("Connecting to account", accounts.get(0, "displayName"), "as there is only one account available");
@@ -711,10 +711,9 @@
 Dialog {
 id: noAccount
 objectName: "noAccountDialog"
-title: i18n.tr("Setup Evernote connection?")
+title: i18n.tr("Sync with Evernote")
 text: i18n.tr("Notes can be stored on this device, or optionally synced with Evernote.") + " "
-  + i18n.tr("In order to synchronize notes with Evernote, an account at Evernote is required.") + " "
-  + i18n.tr("Do you want to setup an account now?")
+  + i18n.tr("To sync with Evernote, you need an Evernote account.")
 
 Connections {
 target: accounts
@@ -729,8 +728,7 @@
 RowLayout {
 Button {
 objectName: "openAccountButton"
-text: i18n.tr("No")
-color: UbuntuColors.red
+text: i18n.tr("Not Now")
 onClicked: {
 PopupUtils.close(noAccount)
 accountService.startAuthentication("@local", null);
@@ -739,7 +737,7 @@
 }
 Button {
 objectName: "openAccountButton"
-text: i18n.tr("Yes")
+text: i18n.tr("Set Up…")
 color: UbuntuColors.green
 onClicked: setup.exec()
 Layout.fillWidth: true

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~renatofilho/ubuntu-calendar-app/bottom-edge into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:762
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1104/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/661/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1104/rebuild

-- 
https://code.launchpad.net/~renatofilho/ubuntu-calendar-app/bottom-edge/+merge/284622
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~renatofilho/ubuntu-calendar-app/week-press-and-hold-create-event into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:763
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1103/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/660/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1103/rebuild

-- 
https://code.launchpad.net/~renatofilho/ubuntu-calendar-app/week-press-and-hold-create-event/+merge/284290
Your team Ubuntu Calendar Developers is requested to review the proposed merge 
of lp:~renatofilho/ubuntu-calendar-app/week-press-and-hold-create-event into 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Bug 1508363] Re: Coordinated migration to UITK 1.3

2016-03-01 Thread Nekhelesh Ramananthan
** Changed in: ubuntu-calendar-app
   Status: In Progress => Fix Released

** Changed in: ubuntu-calendar-app
Milestone: None => 0.5

** Changed in: ubuntu-calendar-app
 Assignee: Alan Pope  (popey) => Renato Araujo Oliveira Filho (renatofilho)

-- 
You received this bug notification because you are a member of Ubuntu
Shorts Developers, which is subscribed to Ubuntu Shorts App.
Matching subscriptions: Ubuntu RSS Reader Bugmail
https://bugs.launchpad.net/bugs/1508363

Title:
  Coordinated migration to UITK 1.3

Status in Canonical System Image:
  In Progress
Status in Ubuntu Music App:
  Fix Released
Status in Ubuntu Notes app:
  Fix Committed
Status in Tagger:
  Fix Released
Status in task-app:
  In Progress
Status in Ubuntu Calculator App:
  Fix Committed
Status in Ubuntu Calendar App:
  Fix Released
Status in Ubuntu Clock App:
  Fix Committed
Status in Ubuntu Document Viewer App:
  Fix Released
Status in Ubuntu Shorts App:
  Fix Committed
Status in Ubuntu Weather App:
  Fix Released
Status in Weather:
  Fix Released
Status in address-book-app package in Ubuntu:
  Fix Committed
Status in calendar-app package in Ubuntu:
  Confirmed
Status in camera-app package in Ubuntu:
  Fix Released
Status in dialer-app package in Ubuntu:
  Fix Committed
Status in gallery-app package in Ubuntu:
  Fix Released
Status in messaging-app package in Ubuntu:
  Fix Committed
Status in music-app package in Ubuntu:
  Confirmed
Status in reminders-app package in Ubuntu:
  Fix Committed
Status in ubuntu-clock-app package in Ubuntu:
  Confirmed
Status in ubuntu-system-settings package in Ubuntu:
  Fix Released
Status in ubuntu-system-settings-online-accounts package in Ubuntu:
  Fix Released
Status in ubuntu-weather-app package in Ubuntu:
  Confirmed
Status in unity8 package in Ubuntu:
  Fix Released
Status in webbrowser-app package in Ubuntu:
  Fix Released

Bug description:
  The migration to 1.3 needs to happen in a coordinated way, so that
  shell and all apps we care of migrate within the same OTA window to
  reduce visual inconsistencies to a minimum.

  This bug is a means to collect all the projects affected and keep tabs
  on what's happening.

  NOTES:
  - The toolkit has the following modules:
 Ubuntu.Components 1.3
 Ubuntu.Components.ListItems 1.3
 Ubuntu.Components.Pickers 1.3
 Ubuntu.Components.Popups 1.3
 Ubuntu.Components.Styles 1.3
 Ubuntu.Components.Themes 1.3
 Ubuntu.Components.Themes.Ambiance 1.3
 Ubuntu.Components.Themes.SuruDark 1.3

  - The following modules were not altered recently:
 Ubuntu.Layouts 1.0
 Ubuntu.PerformanceMetrics 1.0

  - Theme is now theme (lowercase)
  - PageHeadStyle now has a "config" property

  In manifest.json file use:
     "framework": "ubuntu-sdk-15.04.1-qml"

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1508363/+subscriptions

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Nekhelesh Ramananthan
b) General structure of a QML Object is (notice the blank lines),

  Object {
id
objectName

property declarations
signal declarations

JavaScript functions

object properties

child objects

states

transitions
  }
-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Nekhelesh Ramananthan
Review: Needs Fixing code-view & testing

That looks s much better! Awesome work! 

We're almost ready to merge this. Just one last final list of changes to be 
made,

1. The code uses a ListItem { Label { id: header } } to show the Header. I 
suggest we move that as well to using ListItemLayout which will automatically 
take care of eliding the text, positioning it and so on thereby reducing the 
amount of code. I did it and it works nicely. => 
http://paste.ubuntu.com/15258241/

2. Some nit-pick fixes w.r.t qml coding conventions,
a) Color codes should be in full caps..so "#EDEDED" instead of "#ededed". Makes 
it easier to spot them.

b) General structure of a QML Object is (notice the blank lines),

  Object {
id
objectName

property declarations
signal declarations
JavaScript functions
object properties
child objects
states
transitions
  }

Can you ensure all objects (include the AgendaView Page obeys this)?

c) There are a few spelling mistakes in the comments written. For instance, // 
button to be showen when no calendar is selected (onClick will take user to 
list of all calendars) ... where it should be "shown" instead of "showen". 

3. It seems we're importing "import "dateExt.js" as DateExt"..but I don't see 
DateExt being used anywhere in the AgendaView. We can remove it safely.

4. Merge lp:ubuntu-calendar-app. You might get conflicts w.r.t calendar.qml 
since the background color was already changed to white in trunk by me..and you 
did it as well in this branch. If you're unsure how to resolve conflicts, let 
me know.

5. Let's get design opinion of the new agenda-view after the above items are 
fixed before merging this MP.

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector into lp:ubuntu-docviewer-app

2016-03-01 Thread noreply
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector into 
lp:ubuntu-docviewer-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector/+merge/287474
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector into lp:ubuntu-docviewer-app

2016-03-01 Thread Stefano Verzegnassi
The proposal to merge 
lp:~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector into 
lp:ubuntu-docviewer-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/frieza-lok-zoom-selector/+merge/287474
-- 
Your team Ubuntu Document Viewer Developers is subscribed to branch 
lp:ubuntu-docviewer-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:765
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1102/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/659/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1102/rebuild

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread noreply
The proposal to merge lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into 
lp:ubuntu-calendar-app has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~nik90/ubuntu-calendar-app/migrate-listitemlayout/+merge/287309
-- 
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Approve continuous-integration


-- 
https://code.launchpad.net/~nik90/ubuntu-calendar-app/migrate-listitemlayout/+merge/287309
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:764
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1101/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/658/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1101/rebuild

-- 
https://code.launchpad.net/~nik90/ubuntu-calendar-app/migrate-listitemlayout/+merge/287309
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread Nekhelesh Ramananthan
The proposal to merge lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into 
lp:ubuntu-calendar-app has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~nik90/ubuntu-calendar-app/migrate-listitemlayout/+merge/287309
-- 
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:764
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1100/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/657/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1100/rebuild

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread Dan Chapman 
Review: Approve

Looks good to me. Thanks!
-- 
https://code.launchpad.net/~nik90/ubuntu-calendar-app/migrate-listitemlayout/+merge/287309
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:764
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1099/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/656/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1099/rebuild

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp


Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-calendar-app/migrate-listitemlayout into lp:ubuntu-calendar-app

2016-03-01 Thread Dan Chapman 
Review: Needs Fixing

Looking good, just a couple of comments inline :-)

Diff comments:

> === modified file 'CalendarChoicePopup.qml'
> --- CalendarChoicePopup.qml   2016-01-29 14:35:14 +
> +++ CalendarChoicePopup.qml   2016-02-26 12:57:09 +
> @@ -15,125 +15,131 @@
>   * You should have received a copy of the GNU General Public License
>   * along with this program.  If not, see .
>   */
> +
>  import QtQuick 2.4
> +import QtOrganizer 5.0
>  import Ubuntu.Components 1.3
> -import Ubuntu.Components.Popups 1.0
> -import Ubuntu.Components.ListItems 1.0 as ListItem
> -import QtOrganizer 5.0
>  import Ubuntu.SyncMonitor 0.1
> +import Ubuntu.Components.Popups 1.3
>  
>  Page {
> -id: root
> +id: calendarChoicePage
>  objectName: "calendarchoicepopup"
> -property var model;
>  
> -signal collectionUpdated();
> +property var model
> +signal collectionUpdated()
>  
>  visible: false
> -title: i18n.tr("Calendars")
> -
> -head {
> -backAction: Action {
> -text: i18n.tr("Back")
> -iconName: "back"
> -onTriggered: {
> -root.collectionUpdated();
> -pop();
> -}
> -}
> -}
> -
> -head.actions:  Action {
> -objectName: "syncbutton"
> -iconName: "reload"
> -// TRANSLATORS: Please translate this string  to 15 characters only.
> -// Currently ,there is no way we can increase width of action menu 
> currently.
> -text: enabled ? i18n.tr("Sync") : i18n.tr("Syncing")
> -onTriggered: syncMonitor.sync(["calendar"])
> -enabled: (syncMonitor.state !== "syncing")
> -visible: syncMonitor.enabledServices ? 
> syncMonitor.serviceIsEnabled("calendar") : false
> +header: PageHeader {
> +title: i18n.tr("Calendars")
> +leadingActionBar.actions: [
> +Action {
> +text: i18n.tr("Back")
> +iconName: "back"
> +onTriggered: {
> +calendarChoicePage.collectionUpdated();
> +pop();
> +}
> +}
> +]

NitPick: According to the QML coding conventions if a list contains only one 
element then omit the square brackets.

> +trailingActionBar.actions: [
> +Action {
> +objectName: "syncbutton"
> +iconName: "reload"
> +// TRANSLATORS: Please translate this string  to 15 
> characters only.
> +// Currently ,there is no way we can increase width of 
> action menu currently.
> +text: enabled ? i18n.tr("Sync") : i18n.tr("Syncing")
> +onTriggered: syncMonitor.sync(["calendar"])
> +enabled: (syncMonitor.state !== "syncing")
> +visible: syncMonitor.enabledServices ? 
> syncMonitor.serviceIsEnabled("calendar") : false
> +}
> +]
>  }
>  
>  SyncMonitor {
>  id: syncMonitor
>  }
>  
> -ListView {
> +UbuntuListView {

Why the change to UbuntuListView? Afaik the UbuntuListView is geared towards 
handling expandable delegates, which i don't see that happening in this view. 
UbuntuListView just extends upon ListView, so this is probably bringing some 
additional unneeded bindings/memory usage with no real gain.

>  id: calendarsList
> -anchors.fill: parent
> -
> -footer: CalendarListButtonDelegate {
> +
> +anchors { top: calendarChoicePage.header.bottom; left: parent.left; 
> right: parent.right; bottom: parent.bottom }
> +
> +header: ListItem {
>  id: importFromGoogleButton
>  
>  visible: (onlineAccountHelper.status === Loader.Ready)
> -iconSource: "image://theme/google"
> -labelText: i18n.tr("Add online Calendar")
> +height: onlineCalendarLayout.height + divider.height
> +
> +ListItemLayout {
> +id: onlineCalendarLayout
> +title.text: i18n.tr("Add online Calendar")
> +
> +Image {
> +SlotsLayout.position: SlotsLayout.First
> +source: "image://theme/google"
> +width: units.gu(5)
> +height: width
> +}
> +}
> +
>  onClicked: {
>  onlineAccountHelper.item.setupExec()
>  }
>  }
>  
> -model : root.model.getCollections();
> -delegate: ListItem.Standard {
> +model : calendarChoicePage.model.getCollections()
> +currentIndex: -1
> +
> +delegate: ListItem {
>  id: delegateComp
>  objectName: "calendarItem"
>  
> -Rectangle {
> -id: calendarColorCode
> -objectName: "calendarColorCode"
> -
> -width: parent.height - units.gu(2)
> -height: width
> -
> 

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~majster-pl/ubuntu-calendar-app/new-agenda-view into lp:ubuntu-calendar-app

2016-03-01 Thread Jenkins Bot
Review: Needs Fixing continuous-integration

FAILED: Continuous integration, rev:763
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1098/
Executed test runs:
None: https://core-apps-jenkins.ubuntu.com/job/generic-update-mp/655/console

Click here to trigger a rebuild:
https://core-apps-jenkins.ubuntu.com/job/run-ap-tests-ci/1098/rebuild

-- 
https://code.launchpad.net/~majster-pl/ubuntu-calendar-app/new-agenda-view/+merge/287448
Your team Ubuntu Calendar Developers is subscribed to branch 
lp:ubuntu-calendar-app.

-- 
Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers
Post to : ubuntu-touch-coreapps-reviewers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps-reviewers
More help   : https://help.launchpad.net/ListHelp