Merge lp:~knitzsche/ubuntu-rest-scopes/yahoofianance_news.finance_keyword into lp:ubuntu-rest-scopes
| Status: | Merged |
|---|---|
| Approved by: | Bret Barker on 2015-06-23 |
| Approved revision: | 478 |
| Merged at revision: | 478 |
| Proposed branch: | lp:~knitzsche/ubuntu-rest-scopes/yahoofianance_news.finance_keyword |
| Merge into: | lp:ubuntu-rest-scopes |
| Diff against target: |
12 lines (+1/-1) 1 file modified
configs/scopes-info.yaml (+1/-1) |
| To merge this branch: | bzr merge lp:~knitzsche/ubuntu-rest-scopes/yahoofianance_news.finance_keyword |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Facundo Batista | 2015-06-22 | Approve on 2015-06-23 | |
| David Callé (community) | Approve on 2015-06-22 | ||
|
Review via email:
|
|||
Commit Message
Add news.finance keyword to yahoofinance so News scope can still aggregate it after the next update
Description of the Change
Hi,
The news scope will now aggregate by "news.finance" keyword (not just "finance" because it is too generic).
This branch adds "news.finance" to yahoofinance scope.
Thanks,
Kyle
| Kyle Nitzsche (knitzsche) wrote : | # |
Does this need a "top approve" too?
| Kyle Nitzsche (knitzsche) wrote : | # |
I guess I cannot do the merge. Can someone please? This is needed for Yahoo Finance to show up in the News aggregator scope we are about to release.
| Kyle Nitzsche (knitzsche) wrote : | # |
I was asked for a commit message. here it is:
Add news.finance keyword to yahoofinance so News scope can still aggregate it after the next update
| Ubuntu One Auto Pilot (otto-pilot) wrote : | # |
There was a problem validating some authors of the branch. Authors must be either one of the listed Launchpad users, or a member of one of the listed teams on Launchpad.
Persons or Teams:
ubuntu-
Unaccepted Authors:
Kyle Nitzsche <email address hidden>

+1, thanks Kyle