Merge lp://staging/~dobey/pay-service/iap-app-icons into lp://staging/pay-service

Proposed by dobey
Status: Merged
Approved by: Charles Kerr
Approved revision: 113
Merged at revision: 117
Proposed branch: lp://staging/~dobey/pay-service/iap-app-icons
Merge into: lp://staging/pay-service
Diff against target: 192 lines (+65/-16)
7 files modified
CMakeLists.txt (+1/-1)
debian/control (+1/-1)
pay-ui/backend/CMakeLists.txt (+1/-1)
pay-ui/backend/modules/payui/network.cpp (+20/-1)
pay-ui/backend/modules/payui/network.h (+2/-0)
pay-ui/backend/tests/mock_click_server.py (+28/-12)
pay-ui/backend/tests/test_network.cpp (+12/-0)
To merge this branch: bzr merge lp://staging/~dobey/pay-service/iap-app-icons
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+294842@code.staging.launchpad.net

Commit message

Use the ual API to get the icon path for the app for in-app purchases.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

LGTM.

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

to all changes: