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
1=== modified file 'po/com.ubuntu.reminders.pot'
2--- po/com.ubuntu.reminders.pot 2014-08-14 17:17:05 +0000
3+++ po/com.ubuntu.reminders.pot 2014-08-29 12:57:03 +0000
4@@ -8,7 +8,7 @@
5 msgstr ""
6 "Project-Id-Version: com.ubuntu.reminders\n"
7 "Report-Msgid-Bugs-To: \n"
8-"POT-Creation-Date: 2014-08-14 18:16+0100\n"
9+"POT-Creation-Date: 2014-08-29 14:53+0200\n"
10 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
11 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
12 "Language-Team: LANGUAGE <LL@li.org>\n"
13@@ -41,7 +41,7 @@
14 msgstr ""
15
16 #: ../src/app/qml/reminders.qml:320
17-msgid "Please setup an account in the system settings"
18+msgid "Please configure and authorize an Evernote account in System Settings"
19 msgstr ""
20
21 #: ../src/app/qml/reminders.qml:340
22@@ -143,8 +143,8 @@
23
24 #: ../src/app/qml/ui/RemindersPage.qml:108
25 msgid ""
26-"No reminders available. You can create new reminders using the \"Add reminder"
27-"\" button or by setting a reminder when viewing a note."
28+"No reminders available. You can create new reminders by setting a reminder "
29+"when viewing a note."
30 msgstr ""
31
32 #: ../src/app/qml/ui/SearchNotesPage.qml:30
33
34=== modified file 'src/app/qml/reminders.qml'
35--- src/app/qml/reminders.qml 2014-08-06 09:44:18 +0000
36+++ src/app/qml/reminders.qml 2014-08-29 12:57:03 +0000
37@@ -144,7 +144,7 @@
38 onAuthenticated: {
39 if (EvernoteConnection.token && EvernoteConnection.token != reply.AccessToken) {
40 EvernoteConnection.clearToken();
41- }
42+ }
43 EvernoteConnection.token = reply.AccessToken;
44 }
45 onAuthenticationError: {
46@@ -317,7 +317,7 @@
47 id: noAccount
48 objectName: "noAccountDialog"
49 title: i18n.tr("No account available")
50- text: i18n.tr("Please setup an account in the system settings")
51+ text: i18n.tr("Please configure and authorize an Evernote account in System Settings")
52
53 Connections {
54 target: accounts
55
56=== modified file 'src/app/qml/ui/RemindersPage.qml'
57--- src/app/qml/ui/RemindersPage.qml 2014-05-20 15:17:37 +0000
58+++ src/app/qml/ui/RemindersPage.qml 2014-08-29 12:57:03 +0000
59@@ -105,7 +105,7 @@
60 wrapMode: Text.WordWrap
61 horizontalAlignment: Text.AlignHCenter
62 text: notes.error ? notes.error :
63- i18n.tr("No reminders available. You can create new reminders using the \"Add reminder\" button or by setting a reminder when viewing a note.")
64+ i18n.tr("No reminders available. You can create new reminders by setting a reminder when viewing a note.")
65 }
66 }
67 }

Subscribers

People subscribed via source and target branches