Merge lp:~facundo/ubuntu-rest-scopes/new-symbol-lookup into lp:ubuntu-rest-scopes

Proposed by Facundo Batista
Status: Merged
Approved by: Roberto Alsina
Approved revision: 485
Merged at revision: 485
Proposed branch: lp:~facundo/ubuntu-rest-scopes/new-symbol-lookup
Merge into: lp:ubuntu-rest-scopes
Diff against target: 16 lines (+4/-2)
1 file modified
src/scopes/yahoofinance.py (+4/-2)
To merge this branch: bzr merge lp:~facundo/ubuntu-rest-scopes/new-symbol-lookup
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve
Review via email: mp+273404@code.launchpad.net

Commit message

Fixed symbol lookup URL.

Description of the change

Fixed symbol lookup URL.

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/scopes/yahoofinance.py'
2--- src/scopes/yahoofinance.py 2015-03-13 20:16:56 +0000
3+++ src/scopes/yahoofinance.py 2015-10-05 13:40:39 +0000
4@@ -19,8 +19,10 @@
5 BASE_URI = 'http://finance.yahoo.com'
6 STOCKTWITS_URI = 'https://api.stocktwits.com/api/2/trending/symbols.json'
7 SYMBOL_NEWS_URL = "http://finance.yahoo.com/rss/headline?s=%(symbol)s"
8-SYMBOL_LOOKUP_URL = "http://d.yimg.com/autoc.finance.yahoo.com/autoc?" \
9- "query=%(query)s&callback=YAHOO.Finance.SymbolSuggest.ssCallback"
10+SYMBOL_LOOKUP_URL = (
11+ "http://d.yimg.com/autoc.finance.yahoo.com/autoc?"
12+ "query=%(query)s&callback=YAHOO.Finance.SymbolSuggest.ssCallback&region=US&lang=en"
13+)
14
15 # the list of countries that have a standard news URL
16 NEWS_URL_STANDARD = set("ar bg br ca ch de ee es et fi fr hr hu ie it lt lv "

Subscribers

People subscribed via source and target branches