Merge lp://staging/~canonical-ca-hackers/ubuntu-recommender/top-up-recommendations into lp://staging/ubuntu-recommender

Proposed by Łukasz Czyżykowski
Status: Merged
Approved by: Anthony Lenton
Approved revision: 57
Merged at revision: 56
Proposed branch: lp://staging/~canonical-ca-hackers/ubuntu-recommender/top-up-recommendations
Merge into: lp://staging/ubuntu-recommender
Diff against target: 63 lines (+19/-2)
2 files modified
src/recommender/api/handlers.py (+3/-2)
src/recommender/tests/test_api.py (+16/-0)
To merge this branch: bzr merge lp://staging/~canonical-ca-hackers/ubuntu-recommender/top-up-recommendations
Reviewer Review Type Date Requested Status
Anthony Lenton (community) Approve
Review via email: mp+102708@code.staging.launchpad.net

Commit message

Always return RECOMMENDATION_LIMIT number of recommendations from recommend me api call.

Description of the change

Overview
========
Added code to always return RECOMMENDATION_LIMIT number of recommendations from recommend me api call.

To post a comment you must log in.
Revision history for this message
Anthony Lenton (elachuni) wrote :

Thanks lukazs!

Revision history for this message
Anthony Lenton (elachuni) :
review: Approve

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