Ubuntu Rest Scopes

Merge lp://staging/~facundo/ubuntu-rest-scopes/yahoofinance-no-result into lp://staging/ubuntu-rest-scopes

  1. yahoofinance-no-result
  2. Merge into trunk
Proposed by Facundo Batista
Status: Merged
Approved by: Facundo Batista
Approved revision: 489
Merged at revision: 488
Proposed branch: lp://staging/~facundo/ubuntu-rest-scopes/yahoofinance-no-result
Merge into: lp://staging/ubuntu-rest-scopes
Diff against target: 41 lines (+13/-1)
3 files modified
src/scopes/tests/fixtures/yahoofinance-no-result.json (+1/-0)
src/scopes/tests/test_yahoofinance.py (+11/-0)
src/scopes/yahoofinance.py (+1/-1)
To merge this branch: bzr merge lp://staging/~facundo/ubuntu-rest-scopes/yahoofinance-no-result
Reviewer Review Type Date Requested Status
Bret Barker (community) Approve
Review via email: mp+274915@code.staging.launchpad.net

Commit message

Support response with no real result from Yahoo Finance.

Description of the change

Support response with no real result from Yahoo Finance.

To post a comment you must log in.
Revision history for this message
Bret Barker (noise) :
review: Needs Information
Revision history for this message
Facundo Batista (facundo) :
lp://staging/~facundo/ubuntu-rest-scopes/yahoofinance-no-result updated
489. By Facundo Batista

Removed debug code.

Revision history for this message
Bret Barker (noise) wrote :

+1

review: Approve
Revision history for this message
Facundo Batista (facundo) wrote :

Pushed removal of debug code.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches