Merge lp:~savilerow-team/savilerow/unity-scope-tourism-shanghai-mockweather-lp1322337 into lp:~savilerow-team/savilerow/unity-scope-tourism-shanghai-mockweather

Proposed by Scott Sweeny
Status: Merged
Merged at revision: 15
Proposed branch: lp:~savilerow-team/savilerow/unity-scope-tourism-shanghai-mockweather-lp1322337
Merge into: lp:~savilerow-team/savilerow/unity-scope-tourism-shanghai-mockweather
Diff against target: 47 lines (+13/-6)
2 files modified
data/tourism-mockweather.json.in (+6/-6)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~savilerow-team/savilerow/unity-scope-tourism-shanghai-mockweather-lp1322337
Reviewer Review Type Date Requested Status
Chris Wayne (community) Approve
Review via email: mp+221964@code.launchpad.net

Description of the change

Two changes:

1. Populate the "link" fields of results with a non-empty string since the API seems to have become more strict lately.

2. Use the SCOPE_INSTALL_DIR CMake variable in the image paths so they are always in sync with where the scope is installed.

Tested by cwayne on his phone.

To post a comment you must log in.
Revision history for this message
Chris Wayne (cwayne) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'data/tourism-mockweather.json.in'
--- data/tourism-mockweather.json.in 2014-05-01 04:58:15 +0000
+++ data/tourism-mockweather.json.in 2014-06-03 21:26:01 +0000
@@ -2,22 +2,22 @@
2 {2 {
3 "title": "Shanghai, CN Today",3 "title": "Shanghai, CN Today",
4 "subtitle": "10°C",4 "subtitle": "10°C",
5 "link": "",5 "link": "http://www.weather.com",
6 "image": "file:///custom/usr/lib/arm-linux-gnueabihf/unity/scopes/scope-tourism-mockweather/images/overcast.png",6 "image": "file://@SCOPE_INSTALL_DIR@/images/overcast.png",
7 "description": "Minimum temperature: 9°\nMaximum temperature: 14°\nChance of rain: 10%\n Wind: WSW at 16 km/h\nHumidity: 76%\nUV index: 3"7 "description": "Minimum temperature: 9°\nMaximum temperature: 14°\nChance of rain: 10%\n Wind: WSW at 16 km/h\nHumidity: 76%\nUV index: 3"
8 },8 },
9 {9 {
10 "title": "Tomorrow",10 "title": "Tomorrow",
11 "subtitle": "14°C",11 "subtitle": "14°C",
12 "link": "",12 "link": "http://www.weather.com",
13 "image": "file:///custom/usr/lib/arm-linux-gnueabihf/unity/scopes/scope-tourism-mockweather/images/overcast.png",13 "image": "file://@SCOPE_INSTALL_DIR@/images/overcast.png",
14 "description": "Minimum temperature: 8°\nMaximum temperature: 14°\nChance of rain: 0%\n Wind: WSW at 16 km/h\nHumidity: 73%\nUV index: 3"14 "description": "Minimum temperature: 8°\nMaximum temperature: 14°\nChance of rain: 0%\n Wind: WSW at 16 km/h\nHumidity: 73%\nUV index: 3"
15 },15 },
16 {16 {
17 "title": "Tomorrow night",17 "title": "Tomorrow night",
18 "subtitle": "7°C",18 "subtitle": "7°C",
19 "link": "",19 "link": "http://www.weather.com",
20 "image": "file:///custom/usr/lib/arm-linux-gnueabihf/unity/scopes/scope-tourism-mockweather/images/few-clouds-night.png",20 "image": "file://@SCOPE_INSTALL_DIR@/images/few-clouds-night.png",
21 "description": "Minimum temperature: 8°\nMaximum temperature: 14°\nChance of rain: 0%\n Wind: W at 14 km/h\nHumidity: 76%\nUV index: 0"21 "description": "Minimum temperature: 8°\nMaximum temperature: 14°\nChance of rain: 0%\n Wind: W at 14 km/h\nHumidity: 76%\nUV index: 0"
22 }22 }
23]23]
2424
=== modified file 'debian/changelog'
--- debian/changelog 2014-05-13 21:09:26 +0000
+++ debian/changelog 2014-06-03 21:26:01 +0000
@@ -1,3 +1,10 @@
1unity-scope-tourism-shanghai-mockweather (0.7) UNRELEASED; urgency=medium
2
3 * Add fake links to results to cope with API requirements (LP: #1322337)
4 * Use install path CMake variable to populate image paths
5
6 -- Scott Sweeny <scott.sweeny@canonical.com> Tue, 03 Jun 2014 16:38:24 -0400
7
1unity-scope-tourism-shanghai-mockweather (0.6) utopic; urgency=medium8unity-scope-tourism-shanghai-mockweather (0.6) utopic; urgency=medium
29
3 * i18n fixes10 * i18n fixes

Subscribers

People subscribed via source and target branches

to all changes: