Merge lp://staging/~lukasz-czyzykowski/ubuntu-recommender/ubuntu-recommender-client-implicit-feedback into lp://staging/~canonical-ca-hackers/ubuntu-recommender/ubuntu-recommender-client

Proposed by Łukasz Czyżykowski
Status: Merged
Approved by: Anthony Lenton
Approved revision: 8
Merged at revision: 8
Proposed branch: lp://staging/~lukasz-czyzykowski/ubuntu-recommender/ubuntu-recommender-client-implicit-feedback
Merge into: lp://staging/~canonical-ca-hackers/ubuntu-recommender/ubuntu-recommender-client
Diff against target: 22 lines (+12/-0)
1 file modified
sreclient.py (+12/-0)
To merge this branch: bzr merge lp://staging/~lukasz-czyzykowski/ubuntu-recommender/ubuntu-recommender-client-implicit-feedback
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+120171@code.staging.launchpad.net

Commit message

Added implicit_feedback api call

Description of the change

Added implicit_feedback api call

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) wrote :

This looks just fine to me. Thank you, Łukasz!

review: Approve
Revision history for this message
Gary Lasker (gary-lasker) wrote :

P.S. We'll need to formally define the 'action' values. But since we are using only a single "action" currently (that is, a recommended item has been installed), it is not urgent.

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