Merge lp://qastaging/~artmello/gallery-app/gallery-app-qml_content_hub into lp://qastaging/gallery-app

Proposed by Arthur Mello
Status: Merged
Approved by: Bill Filler
Approved revision: 1311
Merged at revision: 1310
Proposed branch: lp://qastaging/~artmello/gallery-app/gallery-app-qml_content_hub
Merge into: lp://qastaging/gallery-app
Diff against target: 906 lines (+181/-429)
10 files modified
CMakeLists.txt (+0/-1)
debian/control (+1/-1)
rc/qml/GalleryApplication.qml (+3/-0)
rc/qml/MainScreen.qml (+51/-38)
rc/qml/PickerScreen.qml (+54/-11)
src/CMakeLists.txt (+0/-2)
src/content-communicator.cpp (+0/-197)
src/content-communicator.h (+0/-69)
src/gallery-application.cpp (+69/-101)
src/gallery-application.h (+3/-9)
To merge this branch: bzr merge lp://qastaging/~artmello/gallery-app/gallery-app-qml_content_hub
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+291426@code.qastaging.launchpad.net

Commit message

Stop using C++ Content Hub API and use QML one instead

Description of the change

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
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks good and works well, thanks!

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