Merge lp:~gang65/ubuntu-weather-app/ubuntu-weather-app-message-text-fix into lp:ubuntu-weather-app

Proposed by Bartosz Kosiorek
Status: Merged
Approved by: Victor Thompson
Approved revision: 147
Merged at revision: 154
Proposed branch: lp:~gang65/ubuntu-weather-app/ubuntu-weather-app-message-text-fix
Merge into: lp:ubuntu-weather-app
Diff against target: 50 lines (+7/-3)
3 files modified
app/components/NetworkErrorStateComponent.qml (+1/-1)
debian/changelog (+4/-0)
po/com.ubuntu.weather.pot (+2/-2)
To merge this branch: bzr merge lp:~gang65/ubuntu-weather-app/ubuntu-weather-app-message-text-fix
Reviewer Review Type Date Requested Status
Victor Thompson Approve
Andrew Hayzen Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+274840@code.launchpad.net

Commit message

Change message text according to Ubuntu UX comment (LP: #1493466)

Description of the change

Change message text according to Ubuntu UX comment (LP: #1493466)

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
Victor Thompson (vthompson) wrote :

I'm not sure we have enough time to get the new string translated prior to releasing the initial version of the app. IMO, we should approve this after the first version is released to allow for this new string to be adequately translated.

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

Now that the initial reboot app is in the store, let's land this and get the new string translated.

Thanks Bartosz!

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
Victor Thompson (vthompson) wrote :

Hi Bartosz, there was a conflict in the merge. Also, Andrew is going to mark the 3.0 app as released, so you'll need to update your changelog entry as well.

review: Needs Fixing
146. By Bartosz Kosiorek

Merge with trunk

147. By Bartosz Kosiorek

Update changelog

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
Andrew Hayzen (ahayzen) wrote :

LGTM :-) Thanks for resolving the conflicts.

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

Great, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/components/NetworkErrorStateComponent.qml'
2--- app/components/NetworkErrorStateComponent.qml 2015-08-28 21:54:22 +0000
3+++ app/components/NetworkErrorStateComponent.qml 2015-10-25 20:14:56 +0000
4@@ -54,7 +54,7 @@
5 anchors {
6 horizontalCenter: parent.horizontalCenter
7 }
8- text: i18n.tr("Ubuntu suggests you check your network settings and try again.")
9+ text: i18n.tr("Please check your network settings and try again.")
10 width: parent.width
11 wrapMode: Text.WordWrap
12 }
13
14=== modified file 'debian/changelog'
15--- debian/changelog 2015-10-24 19:24:48 +0000
16+++ debian/changelog 2015-10-25 20:14:56 +0000
17@@ -1,7 +1,11 @@
18 ubuntu-weather-app (3.1ubuntu1) UNRELEASED; urgency=medium
19
20+ [ Andrew Hayzen ]
21 * Bump version to 3.1
22
23+ [ Bartosz Kosiorek ]
24+ * Change message text according to Ubuntu UX comment (LP: #1493466)
25+
26 -- Andrew Hayzen <ahayzen@gmail.com> Sat, 24 Oct 2015 20:22:31 +0100
27
28 ubuntu-weather-app (3.0ubuntu1) vivid; urgency=medium
29
30=== modified file 'po/com.ubuntu.weather.pot'
31--- po/com.ubuntu.weather.pot 2015-10-22 15:17:02 +0000
32+++ po/com.ubuntu.weather.pot 2015-10-25 20:14:56 +0000
33@@ -8,7 +8,7 @@
34 msgstr ""
35 "Project-Id-Version: ubuntu-weather-app\n"
36 "Report-Msgid-Bugs-To: \n"
37-"POT-Creation-Date: 2015-10-22 16:15+0100\n"
38+"POT-Creation-Date: 2015-10-25 21:12+0100\n"
39 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
40 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
41 "Language-Team: LANGUAGE <LL@li.org>\n"
42@@ -87,7 +87,7 @@
43 msgstr ""
44
45 #: ../app/components/NetworkErrorStateComponent.qml:57
46-msgid "Ubuntu suggests you check your network settings and try again."
47+msgid "Please check your network settings and try again."
48 msgstr ""
49
50 #: ../app/components/NetworkErrorStateComponent.qml:63

Subscribers

People subscribed via source and target branches