Merge lp:~cwayne/news-scope/news-added-cats into lp:~news-scope-team/news-scope/news-scope-outdated

Proposed by Chris Wayne
Status: Merged
Approved by: Kyle Nitzsche
Approved revision: 100
Merged at revision: 100
Proposed branch: lp:~cwayne/news-scope/news-added-cats
Merge into: lp:~news-scope-team/news-scope/news-scope-outdated
Diff against target: 135 lines (+96/-1)
1 file modified
data/child_scopes.json (+96/-1)
To merge this branch: bzr merge lp:~cwayne/news-scope/news-added-cats
Reviewer Review Type Date Requested Status
Kyle Nitzsche (community) Approve
Review via email: mp+259870@code.launchpad.net

Description of the change

Get categories same as they were with old-style news scope

To post a comment you must log in.
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/child_scopes.json'
2--- data/child_scopes.json 2015-05-21 23:41:56 +0000
3+++ data/child_scopes.json 2015-05-22 00:55:30 +0000
4@@ -23,6 +23,11 @@
5 "id": "dept-tech",
6 "root": "False",
7 "_title": "Technology"
8+ },
9+ {
10+ "id": "dept-sports",
11+ "root": "false",
12+ "_title": "Sports"
13 }
14 ]
15 },
16@@ -119,6 +124,36 @@
17 {
18 "scope":
19 {
20+ "id": "com.canonical.elpais_ELPAIS",
21+ "local_id": "elpais-international",
22+ "category_title": "",
23+ "department": "dept-international",
24+ "link_to_child": "false",
25+ "child_department": "http://ep00.epimg.net/rss/internacional/portada.xml",
26+ "surface_template":
27+ {
28+ "components":
29+ {
30+ "art": "art",
31+ "title": "title",
32+ "subtitle": "subtitle",
33+ "summary": "summary",
34+ "emblem": "emblem"
35+ },
36+ "schema-version": 1,
37+ "template":
38+ {
39+ "card-layout": "horizontal",
40+ "card-size": "large",
41+ "collapsed-rows": 0,
42+ "category-layout": "vertical-journal"
43+ }
44+ }
45+ }
46+ },
47+ {
48+ "scope":
49+ {
50 "id": "com.canonical.scopes.bbc_bbc",
51 "local_id": "bbc-national",
52 "category_title": "",
53@@ -149,6 +184,36 @@
54 {
55 "scope":
56 {
57+ "id": "com.canonical.scopes.euronews_euronews",
58+ "local_id": "euronews-national",
59+ "category_title": "",
60+ "department": "dept-national",
61+ "link_to_child": "false",
62+ "child_department": "http://www.euronews.com/_partners/ubuntu/europe_LANGCODE.xml",
63+ "surface_template":
64+ {
65+ "components":
66+ {
67+ "art": "art",
68+ "title": "title",
69+ "subtitle": "subtitle",
70+ "summary": "summary",
71+ "emblem": "emblem"
72+ },
73+ "schema-version": 1,
74+ "template":
75+ {
76+ "card-layout": "horizontal",
77+ "card-size": "large",
78+ "collapsed-rows": 0,
79+ "category-layout": "vertical-journal"
80+ }
81+ }
82+ }
83+ },
84+ {
85+ "scope":
86+ {
87 "id": "com.canonical.elpais_ELPAIS",
88 "local_id": "elpais-national",
89 "category_title": "",
90@@ -179,7 +244,7 @@
91 "keyword":
92 {
93 "keyword": "sports",
94- "_department_title": "Sports",
95+ "department": "dept-sports",
96 "shared_category": "sports",
97 "_shared_category_title": " ",
98 "category_renderer":
99@@ -203,6 +268,36 @@
100 }
101 }
102 },
103+{
104+ "scope":
105+ {
106+ "id": "com.canonical.elpais_ELPAIS",
107+ "local_id": "elpais-sport",
108+ "category_title": "",
109+ "department": "dept-sports",
110+ "link_to_child": "false",
111+ "child_department": "http://ep00.epimg.net/rss/deportes/portada.xml",
112+ "surface_template":
113+ {
114+ "components":
115+ {
116+ "art": "art",
117+ "title": "title",
118+ "subtitle": "subtitle",
119+ "summary": "summary",
120+ "emblem": "emblem"
121+ },
122+ "schema-version": 1,
123+ "template":
124+ {
125+ "card-layout": "horizontal",
126+ "card-size": "large",
127+ "collapsed-rows": 0,
128+ "category-layout": "vertical-journal"
129+ }
130+ }
131+ }
132+ },
133 {
134 "keyword":
135 {

Subscribers

People subscribed via source and target branches

to all changes: