[Ubuntu-touch-coreapps-reviewers] [Bug 1431365] Re: Tests fail at warning dialog A topic with that name already exists

2015-03-16 Thread Alan Pope
It's certainly possible there's a network firewall in place. Have asked someone from Ci to comment. -- You received this bug notification because you are a member of Ubuntu RSS Feed Reader Developers, which is subscribed to Ubuntu RSS Feed Reader App. Matching subscriptions: Ubuntu RSS Reader

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/remove-sorting-close-button into lp:reminders-app

2015-03-19 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~mzanetti/reminders-app/remove-sorting-close-button/+merge/253296 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/reminders-app/fix-1434500 into lp:reminders-app

2015-03-20 Thread Alan Pope
The proposal to merge lp:~popey/reminders-app/fix-1434500 into lp:reminders-app has been updated. Commit Message changed to: Rename user facing parts from Reminders to Notes For more details, see: https://code.launchpad.net/~popey/reminders-app/fix-1434500/+merge/253651 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~flscogna/ubuntu-terminal-app/fix-1431558 into lp:ubuntu-terminal-app

2015-03-19 Thread Alan Pope
The proposal to merge lp:~flscogna/ubuntu-terminal-app/fix-1431558 into lp:ubuntu-terminal-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~flscogna/ubuntu-terminal-app/fix-1431558/+merge/253011 -- Your team Ubuntu Terminal Developers

[Ubuntu-touch-coreapps-reviewers] [Bug 1401718] Re: Core apps should be using the non-dev 14.10 framework

2015-03-09 Thread Alan Pope
** Changed in: ubuntu-weather-app Importance: Undecided = Low ** Changed in: ubuntu-weather-app Status: New = In Progress ** Changed in: ubuntu-weather-app Assignee: (unassigned) = Alan Pope  (popey) -- You received this bug notification because you are a member of Ubuntu File

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-weather-app/fix-1401718 into lp:ubuntu-weather-app

2015-03-09 Thread Alan Pope
@nik90 This merge is for ye olde colourful weather app, not the new one. It's pure QML. -- https://code.launchpad.net/~popey/ubuntu-weather-app/fix-1401718/+merge/252361 Your team Ubuntu Weather Developers is subscribed to branch lp:ubuntu-weather-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters

2015-03-12 Thread Alan Pope
Alan Pope  has proposed merging lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters. Commit message: fix 1401718 Requested reviews: Dropping Letters developers (dropping-letters-devs) Related bugs: Bug #1401718 in Dropping Letters: Core apps should be using the non-dev 14.10

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-typo into lp:ubuntu-terminal-app

2015-03-06 Thread Alan Pope
The proposal to merge lp:~popey/ubuntu-terminal-app/fix-typo into lp:ubuntu-terminal-app has been updated. Commit Message changed to: Fix typo in command bar For more details, see: https://code.launchpad.net/~popey/ubuntu-terminal-app/fix-typo/+merge/252081 -- Your team Ubuntu Terminal

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-clock-app/fix-typo-libya into lp:ubuntu-clock-app

2015-03-06 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-typo-libya/+merge/252079 Your team Ubuntu Clock Developers is subscribed to branch lp:ubuntu-clock-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-clock-app/fix-typo-libya into lp:ubuntu-clock-app

2015-03-06 Thread Alan Pope
The proposal to merge lp:~nik90/ubuntu-clock-app/fix-typo-libya into lp:ubuntu-clock-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-typo-libya/+merge/252079 -- Your team Ubuntu Clock Developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-typo into lp:ubuntu-terminal-app

2015-03-06 Thread Alan Pope
Alan Pope  has proposed merging lp:~popey/ubuntu-terminal-app/fix-typo into lp:ubuntu-terminal-app. Commit message: Fix typo in command bar Requested reviews: Ubuntu Terminal Developers (ubuntu-terminal-dev) For more details, see: https://code.launchpad.net/~popey/ubuntu-terminal-app/fix

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~unity-team/music-app/infographics-translations into lp:music-app

2015-03-10 Thread Alan Pope
The fix for bug 1327419 landed in rtm and vivid, so this can land now? -- https://code.launchpad.net/~unity-team/music-app/infographics-translations/+merge/248251 Your team Music App Developers is subscribed to branch lp:music-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters

2015-03-12 Thread Alan Pope
The proposal to merge lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters has been updated. Commit Message changed to: fix 1401718 For more details, see: https://code.launchpad.net/~popey/dropping-letters/fix-1401718/+merge/252738 -- Your team Dropping Letters developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/fixBlankIcons into lp:ubuntu-calculator-app/reboot

2015-03-24 Thread Alan Pope
The proposal to merge lp:~rpadovani/ubuntu-calculator-app/fixBlankIcons into lp:ubuntu-calculator-app/reboot has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/fixBlankIcons/+merge/252544 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/fixBlankIcons into lp:ubuntu-calculator-app/reboot

2015-03-24 Thread Alan Pope
The proposal to merge lp:~rpadovani/ubuntu-calculator-app/fixBlankIcons into lp:ubuntu-calculator-app/reboot has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/fixBlankIcons/+merge/252544 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app

2015-03-25 Thread Alan Pope
The proposal to merge lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app has been updated. Commit Message changed to: Seems more people than we thought use nano for editing random text files, and having CTRL+X on the CTRL bar is handy for those people. Longer term we should

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app

2015-03-25 Thread Alan Pope
Alan Pope  has proposed merging lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app. Requested reviews: Ubuntu Terminal Developers (ubuntu-terminal-dev) For more details, see: https://code.launchpad.net/~popey/ubuntu-terminal-app/fix-1430660/+merge/254045 -- Your team

[Ubuntu-touch-coreapps-reviewers] [Bug 1413821] Re: Music files are imported into the Music app, rather than simply opened

2015-03-25 Thread Alan Pope
** Changed in: ubuntu-filemanager-app Status: Won't Fix = 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

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/improveFavouriteTextfield into lp:ubuntu-calculator-app/reboot

2015-03-30 Thread Alan Pope
Review: Approve Seems I had some crap on my device. It's working fine now! -- https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/improveFavouriteTextfield/+merge/254001 Your team Ubuntu Calculator Developers is subscribed to branch lp:ubuntu-calculator-app/reboot. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app

2015-03-29 Thread Alan Pope
Now we have a merge for a full nano profile, it seems less useful to have this. -- https://code.launchpad.net/~popey/ubuntu-terminal-app/fix-1430660/+merge/254045 Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~popey/ubuntu-terminal-app/fix-1430660 into

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app

2015-03-29 Thread Alan Pope
The proposal to merge lp:~popey/ubuntu-terminal-app/fix-1430660 into lp:ubuntu-terminal-app has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~popey/ubuntu-terminal-app/fix-1430660/+merge/254045 -- Your team Ubuntu Terminal

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/workaround-fix-1432412 into lp:ubuntu-docviewer-app

2015-03-26 Thread Alan Pope
Review: Approve Tested on bq device and works fine. We may need to re-visit on meizu device (which runs vivid, and a newer kernel) because it seems to never spin more than 1 core even under heavy load in this app. For now, I say approve as bq device is only one shipping, and we can analyse

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/qmltest2 into lp:reminders-app

2015-03-26 Thread Alan Pope
CI have a card for this which they're tracking. https://trello.com/c/2f8qfYun/87-vanguard-reminders-app-ci-failures-due-to-https-bugs-launchpad-net-ubuntu-source-llvm-toolchain-snapshot-bug-1389729 No solution proposed though. --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~manglasape/ubuntu-terminal-app/nanoProfile into lp:ubuntu-terminal-app

2015-03-27 Thread Alan Pope
The proposal to merge lp:~manglasape/ubuntu-terminal-app/nanoProfile into lp:ubuntu-terminal-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~manglasape/ubuntu-terminal-app/nanoProfile/+merge/254156 -- Your team Ubuntu Terminal

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/fixedTests into lp:ubuntu-calculator-app/reboot

2015-03-02 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/fixedTests/+merge/251418 Your team Ubuntu Calculator Developers is subscribed to branch lp:ubuntu-calculator-app/reboot. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2 into lp:ubuntu-docviewer-app

2015-03-03 Thread Alan Pope
Review: Needs Fixing Seems launching a pdf from another application via content hub broke. * Open browser * Search for pdf + weather * Click a random pdf file * Content hub popup, select docviewer * Wait for download * Wait for popup, choose 'Open' * Docviewer launches, but document list is

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2 into lp:ubuntu-docviewer-app

2015-03-03 Thread Alan Pope
Review: Approve I didn't realise this was an intentional design decision based on what the gallery uses. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/document-hub2/+merge/251166 Your team Ubuntu Document Viewer Developers is subscribed to branch

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/fix-tags-crash into lp:reminders-app

2015-03-04 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~mzanetti/reminders-app/fix-tags-crash/+merge/251812 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1428248 into lp:ubuntu-docviewer-app

2015-03-04 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1428248/+merge/251811 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

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/disable-push-notifications into lp:reminders-app

2015-03-05 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~mzanetti/reminders-app/disable-push-notifications/+merge/251985 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ricsipontaz/ubuntu-filemanager-app/translation-fix into lp:ubuntu-filemanager-app

2015-03-04 Thread Alan Pope
The proposal to merge lp:~ricsipontaz/ubuntu-filemanager-app/translation-fix into lp:ubuntu-filemanager-app has been updated. Commit Message changed to: Make the bottomedge's Places string translatable with i18n.tr() For more details, see:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ricsipontaz/ubuntu-docviewer-app/translation-fix-and-template-update into lp:ubuntu-docviewer-app

2015-03-04 Thread Alan Pope
The proposal to merge lp:~ricsipontaz/ubuntu-docviewer-app/translation-fix-and-template-update into lp:ubuntu-docviewer-app has been updated. Status: Needs review = Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~ricsipontaz/ubuntu-docviewer-app/translation-fix-and-template-update into lp:ubuntu-docviewer-app

2015-03-04 Thread Alan Pope
Review: Approve Thanks for the contribution! -- https://code.launchpad.net/~ricsipontaz/ubuntu-docviewer-app/translation-fix-and-template-update/+merge/251727 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/LiveEventModification into lp:ubuntu-calendar-app

2015-03-05 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/LiveEventModification into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/LiveEventModification/+merge/247711 -- Your

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/conflict-handling into lp:reminders-app

2015-03-26 Thread Alan Pope
Review: Approve This worked really well! http://people.canonical.com/~alan/screenshots/device-2015-03-26-180210.png -- https://code.launchpad.net/~mzanetti/reminders-app/conflict-handling/+merge/253002 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/add-night-mode into lp:ubuntu-docviewer-app

2015-03-26 Thread Alan Pope
A few observations:- * Night mode only works in the document, and if you go back to the document list it switches back to 'day' mode. Intentional? * Magazines look funny when inverted. Is this common in other doc/ebook readers?

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent into lp:ubuntu-calendar-app

2015-03-26 Thread Alan Pope
Tapping created an event at 21:26, I think it should create an event which locks to the nearest 30 minutes. The event is created, and then two more button presses are required (tap event, tap edit) to edit the event to change the title from Untitled. Would it make more sense to go directly to

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-clock-app/improve-setting-listitem into lp:ubuntu-clock-app

2015-04-14 Thread Alan Pope
Review: Approve Looks good to me, testing on krillin. Nice work! -- https://code.launchpad.net/~nik90/ubuntu-clock-app/improve-setting-listitem/+merge/252930 Your team Ubuntu Clock Developers is subscribed to branch lp:ubuntu-clock-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1418648 into lp:ubuntu-docviewer-app

2015-04-24 Thread Alan Pope
I presume the crash is to be fixed by https://bugs.launchpad.net/ubuntu-docviewer-app/+bug/1444971 but the issue here is that the popup isn't dismissed, and the page doesn't appear. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1418648/+merge/256354 Your team

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1444972 into lp:ubuntu-docviewer-app

2015-04-24 Thread Alan Pope
Review: Approve Great! Works well, thanks Stefano. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1444972/+merge/256486 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-temporarly-result-fix into lp:ubuntu-calculator-app

2015-04-27 Thread Alan Pope
Are you re-running and expecting it to pass? Seems to fail randomly? -- https://code.launchpad.net/~gang65/ubuntu-calculator-app/ubuntu-calculator-app-temporarly-result-fix/+merge/257447 Your team Ubuntu Calculator Developers is subscribed to branch lp:ubuntu-calculator-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-temporarly-result-fix into lp:ubuntu-calculator-app

2015-04-27 Thread Alan Pope
The proposal to merge lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-temporarly-result-fix into lp:ubuntu-calculator-app has been updated. Status: Needs review = Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-temporarly-result-fix into lp:ubuntu-calculator-app

2015-04-27 Thread Alan Pope
Have run the calculation manually on the phone as per this test:- def test_operation_on_large_numbers(self): self.app.main_view.insert('999*999=') self._assert_result_is(u'9.98e+21') self._assert_history_contains(u'999×999='

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/ch-imported-documents-name into lp:ubuntu-docviewer-app

2015-04-27 Thread Alan Pope
Eek! I see. Well spotted. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/ch-imported-documents-name/+merge/256841 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/that-darn-focus into lp:reminders-app

2015-04-27 Thread Alan Pope
Review: Approve Yup, that works. -- https://code.launchpad.net/~mzanetti/reminders-app/that-darn-focus/+merge/257554 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/that-darn-focus into lp:reminders-app

2015-04-28 Thread Alan Pope
The proposal to merge lp:~mzanetti/reminders-app/that-darn-focus into lp:reminders-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~mzanetti/reminders-app/that-darn-focus/+merge/257554 -- Your team Ubuntu Reminders app developers is

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/still-that-focus into lp:reminders-app

2015-05-07 Thread Alan Pope
The proposal to merge lp:~mzanetti/reminders-app/still-that-focus into lp:reminders-app has been updated. Commit Message changed to: Fix 1452907 For more details, see: https://code.launchpad.net/~mzanetti/reminders-app/still-that-focus/+merge/258566 -- Your team Ubuntu Reminders app

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/still-that-focus into lp:reminders-app

2015-05-07 Thread Alan Pope
Review: Approve Fixes the issues, thanks! -- https://code.launchpad.net/~mzanetti/reminders-app/still-that-focus/+merge/258566 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/fix-renderwidth into lp:reminders-app

2015-05-08 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~mzanetti/reminders-app/fix-renderwidth/+merge/258564 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carla-sella/ubuntu-docviewer-app/test-toc into lp:ubuntu-docviewer-app

2015-05-15 Thread Alan Pope
The proposal to merge lp:~carla-sella/ubuntu-docviewer-app/test-toc into lp:ubuntu-docviewer-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~carla-sella/ubuntu-docviewer-app/test-toc/+merge/258082 -- Your team Ubuntu Document Viewer

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fix-1418648 into lp:ubuntu-docviewer-app

2015-05-15 Thread Alan Pope
Any update on this Stefano? -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fix-1418648/+merge/256354 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-15 Thread Alan Pope
(proposing so I can comment) Tested this on bq device running vivid. Put spread documents between ~/Documents and /media/phablet/2541-1C26/Documents/ $ ls /media/*/*/[Dd][Oo][Cc][Uu][Mm][Ee][Nn][Tt][Ss]/ Manual_Aquaris_E4.1.5_ubuntu_EN.pdf Manual_Aquaris_E4.5_ubuntu_EN.pdf Manual_Aquaris_E4

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/plugins-renaming into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope
Review: Approve LGTM! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/plugins-renaming/+merge/256353 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope
Review: Approve It works! I can see and open documents from both the SD card and the internet storage now. phablet@ubuntu-phablet:~/.cache/upstart$ tail -f application-click-com.ubuntu.docviewer_docviewer_0.3.latest.log /var/lib/apport /etc/default/apport /var/lib/urfkill /var/lib/misc

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/sdcard-read-support into lp:ubuntu-docviewer-app has been updated. Commit Message changed to: SD Card support For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/advanced-text-editor into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope
Review: Needs Fixing This is looking great for viewing small text files. A few issues:- * Large text files can lock up / crash the app. By large I mean 910kB such as https://www.gutenberg.org/cache/epub/48984/pg48984.txt * First line number gets messed up with large files

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/EventColorOnMonth into lp:ubuntu-calendar-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/EventColorOnMonth into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EventColorOnMonth/+merge/259321 -- Your team

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/EventColorOnMonth into lp:ubuntu-calendar-app

2015-05-18 Thread Alan Pope
Review: Approve LGTM. http://people.canonical.com/~alan/screenshots/device-2015-05-18-144814.png -- https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/EventColorOnMonth/+merge/259321 Your team Ubuntu Calendar Developers is subscribed to branch lp:ubuntu-calendar-app. -- Mailing

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/plugins-renaming into lp:ubuntu-docviewer-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/plugins-renaming 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/plugins-renaming/+merge/256353 --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~flscogna/ubuntu-terminal-app/profile-selection into lp:ubuntu-terminal-app

2015-05-19 Thread Alan Pope
Are you planning to add the grey icon in this merge? If so, when can we expect that? -- https://code.launchpad.net/~flscogna/ubuntu-terminal-app/profile-selection/+merge/255077 Your team Ubuntu Terminal Developers is subscribed to branch lp:ubuntu-terminal-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-07 into lp:ubuntu-filemanager-app

2015-05-19 Thread Alan Pope
With those changes, is this ready to land now? -- https://code.launchpad.net/~carlos-mazieri/ubuntu-filemanager-app/samba-browsing-07/+merge/252976 Your team Ubuntu File Manager Developers is subscribed to branch lp:ubuntu-filemanager-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters

2015-05-19 Thread Alan Pope
The proposal to merge lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~popey/dropping-letters/fix-1401718/+merge/252738 -- Your team Dropping Letters developers is requested

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent into lp:ubuntu-calendar-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent/+merge/254175 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carla-sella/ubuntu-calendar-app/fix-dayview-default-view into lp:ubuntu-calendar-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~carla-sella/ubuntu-calendar-app/fix-dayview-default-view into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~carla-sella/ubuntu-calendar-app/fix-dayview-default-view/+merge/257814 --

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/music-app/refactor-readme-update into lp:music-app

2015-05-18 Thread Alan Pope
The proposal to merge lp:~vthompson/music-app/refactor-readme-update into lp:music-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~vthompson/music-app/refactor-readme-update/+merge/259061 -- Your team Music App Developers is

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/music-app/refactor-readme-update into lp:music-app

2015-05-18 Thread Alan Pope
Doing the re-approve dance. -- https://code.launchpad.net/~vthompson/music-app/refactor-readme-update/+merge/259061 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:~verzegnassi-stefano/ubuntu-docviewer-app/landscape-support into lp:ubuntu-docviewer-app

2015-04-14 Thread Alan Pope
Review: Approve Works nicely on Meizu device. Thanks Stefano. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/landscape-support/+merge/256146 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler into lp:ubuntu-docviewer-app

2015-04-13 Thread Alan Pope
Merge conflict it seems.. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler/+merge/255837 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~flscogna/ubuntu-terminal-app/profile-selection into lp:ubuntu-terminal-app

2015-04-13 Thread Alan Pope
I prefer 2). If someone doesn't want to 'waste' the screen real-estate used by the layout, so switches them all off, they'll be disappointed that they don't gain space IMO. If we force a profile to be active, we're forcing that 'waste' of space for users who don't need it. I would default

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/Monthly-YearlyEvent into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/Monthly-YearlyEvent into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/Monthly-YearlyEvent/+merge/255919 -- Your team

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/Monthly-YearlyEvent into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
Review: Approve Tested on Krillin. Nice one Kunal, thanks for the fix. -- https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/Monthly-YearlyEvent/+merge/255919 Your team Ubuntu Calendar Developers is subscribed to branch lp:ubuntu-calendar-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/WeekNumber into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/WeekNumber into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/WeekNumber/+merge/255078 -- Your team Ubuntu Calendar

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carla-sella/ubuntu-calendar-app/weekview-testing-stubs-completed into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
Review: Approve Looks great, thanks Carla! -- https://code.launchpad.net/~carla-sella/ubuntu-calendar-app/weekview-testing-stubs-completed/+merge/254495 Your team Ubuntu Calendar Developers is subscribed to branch lp:ubuntu-calendar-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdf-toc-improvements into lp:ubuntu-docviewer-app

2015-04-12 Thread Alan Pope
Review: Approve This works really nicely, thanks Stefano! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/pdf-toc-improvements/+merge/255518 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/TapToCreateEvent/+merge/254175 -- Your team Ubuntu

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~carla-sella/ubuntu-calendar-app/weekview-testing-stubs-completed into lp:ubuntu-calendar-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~carla-sella/ubuntu-calendar-app/weekview-testing-stubs-completed into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler into lp:ubuntu-docviewer-app

2015-04-12 Thread Alan Pope
Review: Approve This works really nicely, thanks Stefano. -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler/+merge/255837 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler into lp:ubuntu-docviewer-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/adv-import-handler 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/adv-import-handler/+merge/255837

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fullscreen-support into lp:ubuntu-docviewer-app

2015-04-12 Thread Alan Pope
Review: Approve LGTM! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/fullscreen-support/+merge/255856 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/fullscreen-support into lp:ubuntu-docviewer-app

2015-04-12 Thread Alan Pope
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/fullscreen-support 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/fullscreen-support/+merge/255856

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~popey/dropping-letters/fix-1401718 into lp:dropping-letters

2015-04-15 Thread Alan Pope
Victor, could you please review this tiny change. Doing a spot of housework. -- https://code.launchpad.net/~popey/dropping-letters/fix-1401718/+merge/252738 Your team Dropping Letters developers is requested to review the proposed merge of lp:~popey/dropping-letters/fix-1401718 into

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~nik90/ubuntu-clock-app/fix-location-error-msg into lp:ubuntu-clock-app

2015-04-15 Thread Alan Pope
Review: Approve Tested as per instructions on krillin. This is a very nice change, thanks Nekhelesh! -- https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-location-error-msg/+merge/256230 Your team Ubuntu Clock Developers is subscribed to branch lp:ubuntu-clock-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~flscogna/ubuntu-terminal-app/profile-selection into lp:ubuntu-terminal-app

2015-04-15 Thread Alan Pope
Ok, you make a good argument. Let's go with your plan. -- https://code.launchpad.net/~flscogna/ubuntu-terminal-app/profile-selection/+merge/255077 Your team Ubuntu Terminal Developers is subscribed to branch lp:ubuntu-terminal-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/pdfview-show-hide-header into lp:ubuntu-docviewer-app

2015-04-07 Thread Alan Pope
Review: Approve This works really nicely! Thanks Stefano! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/pdfview-show-hide-header/+merge/255076 Your team Ubuntu Document Viewer Developers is subscribed to branch lp:ubuntu-docviewer-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/waitUntilOskIsHidden into lp:ubuntu-calculator-app/reboot

2015-04-07 Thread Alan Pope
Review: Approve Tested this on my krillin and it works well! -- https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/waitUntilOskIsHidden/+merge/254875 Your team Ubuntu Calculator Developers is subscribed to branch lp:ubuntu-calculator-app/reboot. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/add-night-mode into lp:ubuntu-docviewer-app

2015-04-02 Thread Alan Pope
The proposal to merge lp:~verzegnassi-stefano/ubuntu-docviewer-app/add-night-mode 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/add-night-mode/+merge/254255 -- Your

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~verzegnassi-stefano/ubuntu-docviewer-app/add-night-mode into lp:ubuntu-docviewer-app

2015-04-02 Thread Alan Pope
Review: Approve Ok! -- https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/add-night-mode/+merge/254255 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

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/CalendarIcon into lp:ubuntu-calendar-app

2015-04-02 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/CalendarIcon/+merge/254178 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] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/CalendarIcon into lp:ubuntu-calendar-app

2015-04-02 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/CalendarIcon into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/CalendarIcon/+merge/254178 -- Your team Ubuntu

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~rpadovani/ubuntu-calculator-app/setDefaultIcon into lp:ubuntu-calculator-app/reboot

2015-04-09 Thread Alan Pope
Review: Approve That fixed it, thanks! http://people.canonical.com/~alan/screenshots/device-2015-04-09-181327.png -- https://code.launchpad.net/~rpadovani/ubuntu-calculator-app/setDefaultIcon/+merge/255717 Your team Ubuntu Calculator Developers is subscribed to branch

[Ubuntu-touch-coreapps-reviewers] [Bug 1377638] Re: [All Apps] Core apps should use the new splash screen features

2015-04-09 Thread Alan Pope
** Changed in: ubuntu-calendar-app Status: Fix Committed = Fix Released -- 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, Ubuntu RSS Reader

[Ubuntu-touch-coreapps-reviewers] [Bug 1401718] Re: Core apps should be using the non-dev 14.10 framework

2015-04-09 Thread Alan Pope
** Changed in: ubuntu-calendar-app Status: Fix Committed = Fix Released -- 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, Ubuntu RSS Reader

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/confirm-autocorrection-on-save into lp:reminders-app

2015-04-09 Thread Alan Pope
Review: Approve Tested on krillin. Fixes the issue for me. -- https://code.launchpad.net/~mzanetti/reminders-app/confirm-autocorrection-on-save/+merge/255730 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list:

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~mzanetti/reminders-app/fix-autocorrection-formatting into lp:reminders-app

2015-04-09 Thread Alan Pope
Review: Approve Looks good! Tested on krillin. -- https://code.launchpad.net/~mzanetti/reminders-app/fix-autocorrection-formatting/+merge/255736 Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app. -- Mailing list:

[Ubuntu-touch-coreapps-reviewers] [Bug 1456833] Re: Failure to remove applications

2015-05-20 Thread Alan Pope
*** This bug is a duplicate of bug 1358294 *** https://bugs.launchpad.net/bugs/1358294 ** This bug has been marked a duplicate of bug 1358294 App .config not removed when app uninstalled -- You received this bug notification because you are a member of Ubuntu File Manager Developers,

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight into lp:ubuntu-calendar-app

2015-05-20 Thread Alan Pope
The autopilot test seems to send the mouse to the wrong location on screen. http://91.189.93.70:8080/job/generic-mediumtests-utopic/2774/artifact/calendar_app.tests.test_weekview.TestWeekView.test_selecting_a_day_switches_to_day_view.ogv --

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight into lp:ubuntu-calendar-app

2015-05-20 Thread Alan Pope
Review: Approve LGTM -- https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight/+merge/259111 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] [Merge] lp:~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight into lp:ubuntu-calendar-app

2015-05-20 Thread Alan Pope
The proposal to merge lp:~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight into lp:ubuntu-calendar-app has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~pkunal-parmar/ubuntu-calendar-app/WeekViewHighlight/+merge/259111 -- Your team

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~vthompson/ubuntu-weather-app/reboot-match-click-debian-version-numbers into lp:ubuntu-weather-app/reboot

2015-06-02 Thread Alan Pope
Seems there's an issue with the debian config, unrelated to this change, but the deb errors here:- E: ubuntu-weather-app source: source-is-missing app/data/CitiesList.js -- https://code.launchpad.net/~vthompson/ubuntu-weather-app/reboot-match-click-debian-version-numbers/+merge/260780 Your

Re: [Ubuntu-touch-coreapps-reviewers] [Merge] lp:~dholbach/help-app/fix-QA-token into lp:help-app

2015-06-05 Thread Alan Pope
Review: Approve Looks good to me! -- https://code.launchpad.net/~dholbach/help-app/fix-QA-token/+merge/261177 Your team Ubuntu Help app developers is subscribed to branch lp:help-app. -- Mailing list: https://launchpad.net/~ubuntu-touch-coreapps-reviewers Post to :

  1   2   3   4   5   >