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
=== modified file 'app/components/NetworkErrorStateComponent.qml'
--- app/components/NetworkErrorStateComponent.qml 2015-08-28 21:54:22 +0000
+++ app/components/NetworkErrorStateComponent.qml 2015-10-25 20:14:56 +0000
@@ -54,7 +54,7 @@
54 anchors {54 anchors {
55 horizontalCenter: parent.horizontalCenter55 horizontalCenter: parent.horizontalCenter
56 }56 }
57 text: i18n.tr("Ubuntu suggests you check your network settings and try again.")57 text: i18n.tr("Please check your network settings and try again.")
58 width: parent.width58 width: parent.width
59 wrapMode: Text.WordWrap59 wrapMode: Text.WordWrap
60 }60 }
6161
=== modified file 'debian/changelog'
--- debian/changelog 2015-10-24 19:24:48 +0000
+++ debian/changelog 2015-10-25 20:14:56 +0000
@@ -1,7 +1,11 @@
1ubuntu-weather-app (3.1ubuntu1) UNRELEASED; urgency=medium1ubuntu-weather-app (3.1ubuntu1) UNRELEASED; urgency=medium
22
3 [ Andrew Hayzen ]
3 * Bump version to 3.14 * Bump version to 3.1
45
6 [ Bartosz Kosiorek ]
7 * Change message text according to Ubuntu UX comment (LP: #1493466)
8
5 -- Andrew Hayzen <ahayzen@gmail.com> Sat, 24 Oct 2015 20:22:31 +01009 -- Andrew Hayzen <ahayzen@gmail.com> Sat, 24 Oct 2015 20:22:31 +0100
610
7ubuntu-weather-app (3.0ubuntu1) vivid; urgency=medium11ubuntu-weather-app (3.0ubuntu1) vivid; urgency=medium
812
=== modified file 'po/com.ubuntu.weather.pot'
--- po/com.ubuntu.weather.pot 2015-10-22 15:17:02 +0000
+++ po/com.ubuntu.weather.pot 2015-10-25 20:14:56 +0000
@@ -8,7 +8,7 @@
8msgstr ""8msgstr ""
9"Project-Id-Version: ubuntu-weather-app\n"9"Project-Id-Version: ubuntu-weather-app\n"
10"Report-Msgid-Bugs-To: \n"10"Report-Msgid-Bugs-To: \n"
11"POT-Creation-Date: 2015-10-22 16:15+0100\n"11"POT-Creation-Date: 2015-10-25 21:12+0100\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"
@@ -87,7 +87,7 @@
87msgstr ""87msgstr ""
8888
89#: ../app/components/NetworkErrorStateComponent.qml:5789#: ../app/components/NetworkErrorStateComponent.qml:57
90msgid "Ubuntu suggests you check your network settings and try again."90msgid "Please check your network settings and try again."
91msgstr ""91msgstr ""
9292
93#: ../app/components/NetworkErrorStateComponent.qml:6393#: ../app/components/NetworkErrorStateComponent.qml:63

Subscribers

People subscribed via source and target branches