Merge lp:~rpadovani/reminders-app/updateStrings into lp:reminders-app

Proposed by Riccardo Padovani
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 226
Merged at revision: 235
Proposed branch: lp:~rpadovani/reminders-app/updateStrings
Merge into: lp:reminders-app
Diff against target: 67 lines (+7/-7)
3 files modified
po/com.ubuntu.reminders.pot (+4/-4)
src/app/qml/reminders.qml (+2/-2)
src/app/qml/ui/RemindersPage.qml (+1/-1)
To merge this branch: bzr merge lp:~rpadovani/reminders-app/updateStrings
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Victor Thompson (community) Approve
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Review via email: mp+231125@code.launchpad.net

Commit message

Update string for no account advice, removed reference to add reminder button

Description of the change

- Updated string for no account advice, see #1340635
- Removed reference to add reminder button in reminder page, see #1356966

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

Looks good, thanks!

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Victor Thompson (vthompson) wrote :

Please use the en-US spelling of "authorized", which can then be translated into en-GB as "authorised".

review: Needs Fixing
Revision history for this message
Riccardo Padovani (rpadovani) wrote :

Fixed, thanks.

Of course authorised was suggested by @popey :P

Revision history for this message
Victor Thompson (vthompson) wrote :

Thanks Riccardo! You'll also want to update the POT file.

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Riccardo Padovani (rpadovani) wrote :

Oh, man! What a distraction! Fixed

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Victor Thompson (vthompson) wrote :

LGTM!

There appears to be an autopilot issue, unrelated to this change, however.

review: Approve
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

The error in the AP tests occurs in trunk, and is currently being worked;

(run.py:4309): GLib-GIO-CRITICAL **: g_simple_async_result_complete_in_idle: assertion 'G_IS_SIMPLE_ASYNC_RESULT (simple)' failed

I guess since it's happening in jenkins as well, the issue is even more urgent as it seems it will block mp's from landing.

Revision history for this message
Leo Arias (elopio) wrote :

I've found that this solves the error: http://paste.ubuntu.com/8195070/
I do not understand why. On the Glib MainLoop docs I can't find anything about a required wait after starting to run it. I think Martin Pitt knows a lot about GLib, so I will send him an email.

This is reported here: https://bugs.launchpad.net/reminders-app/+bug/1363604

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'po/com.ubuntu.reminders.pot'
--- po/com.ubuntu.reminders.pot 2014-08-14 17:17:05 +0000
+++ po/com.ubuntu.reminders.pot 2014-08-29 12:57:03 +0000
@@ -8,7 +8,7 @@
8msgstr ""8msgstr ""
9"Project-Id-Version: com.ubuntu.reminders\n"9"Project-Id-Version: com.ubuntu.reminders\n"
10"Report-Msgid-Bugs-To: \n"10"Report-Msgid-Bugs-To: \n"
11"POT-Creation-Date: 2014-08-14 18:16+0100\n"11"POT-Creation-Date: 2014-08-29 14:53+0200\n"
12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
14"Language-Team: LANGUAGE <LL@li.org>\n"14"Language-Team: LANGUAGE <LL@li.org>\n"
@@ -41,7 +41,7 @@
41msgstr ""41msgstr ""
4242
43#: ../src/app/qml/reminders.qml:32043#: ../src/app/qml/reminders.qml:320
44msgid "Please setup an account in the system settings"44msgid "Please configure and authorize an Evernote account in System Settings"
45msgstr ""45msgstr ""
4646
47#: ../src/app/qml/reminders.qml:34047#: ../src/app/qml/reminders.qml:340
@@ -143,8 +143,8 @@
143143
144#: ../src/app/qml/ui/RemindersPage.qml:108144#: ../src/app/qml/ui/RemindersPage.qml:108
145msgid ""145msgid ""
146"No reminders available. You can create new reminders using the \"Add reminder"146"No reminders available. You can create new reminders by setting a reminder "
147"\" button or by setting a reminder when viewing a note."147"when viewing a note."
148msgstr ""148msgstr ""
149149
150#: ../src/app/qml/ui/SearchNotesPage.qml:30150#: ../src/app/qml/ui/SearchNotesPage.qml:30
151151
=== modified file 'src/app/qml/reminders.qml'
--- src/app/qml/reminders.qml 2014-08-06 09:44:18 +0000
+++ src/app/qml/reminders.qml 2014-08-29 12:57:03 +0000
@@ -144,7 +144,7 @@
144 onAuthenticated: {144 onAuthenticated: {
145 if (EvernoteConnection.token && EvernoteConnection.token != reply.AccessToken) {145 if (EvernoteConnection.token && EvernoteConnection.token != reply.AccessToken) {
146 EvernoteConnection.clearToken();146 EvernoteConnection.clearToken();
147 } 147 }
148 EvernoteConnection.token = reply.AccessToken;148 EvernoteConnection.token = reply.AccessToken;
149 }149 }
150 onAuthenticationError: {150 onAuthenticationError: {
@@ -317,7 +317,7 @@
317 id: noAccount317 id: noAccount
318 objectName: "noAccountDialog"318 objectName: "noAccountDialog"
319 title: i18n.tr("No account available")319 title: i18n.tr("No account available")
320 text: i18n.tr("Please setup an account in the system settings")320 text: i18n.tr("Please configure and authorize an Evernote account in System Settings")
321321
322 Connections {322 Connections {
323 target: accounts323 target: accounts
324324
=== modified file 'src/app/qml/ui/RemindersPage.qml'
--- src/app/qml/ui/RemindersPage.qml 2014-05-20 15:17:37 +0000
+++ src/app/qml/ui/RemindersPage.qml 2014-08-29 12:57:03 +0000
@@ -105,7 +105,7 @@
105 wrapMode: Text.WordWrap105 wrapMode: Text.WordWrap
106 horizontalAlignment: Text.AlignHCenter106 horizontalAlignment: Text.AlignHCenter
107 text: notes.error ? notes.error :107 text: notes.error ? notes.error :
108 i18n.tr("No reminders available. You can create new reminders using the \"Add reminder\" button or by setting a reminder when viewing a note.")108 i18n.tr("No reminders available. You can create new reminders by setting a reminder when viewing a note.")
109 }109 }
110 }110 }
111}111}

Subscribers

People subscribed via source and target branches