Merge lp://staging/~michael-sheldon/content-hub/fix-1383732-rtm into lp://staging/content-hub/rtm-14.09

Proposed by Michael Sheldon
Status: Merged
Approved by: Ken VanDine
Approved revision: 139
Merged at revision: 139
Proposed branch: lp://staging/~michael-sheldon/content-hub/fix-1383732-rtm
Merge into: lp://staging/content-hub/rtm-14.09
Diff against target: 138 lines (+29/-2)
7 files modified
import/Ubuntu/Content/contentpeermodel.cpp (+1/-0)
import/Ubuntu/Content/contenttype.cpp (+6/-0)
import/Ubuntu/Content/contenttype.h (+2/-1)
include/com/ubuntu/content/type.h (+1/-0)
src/com/ubuntu/content/service/com.ubuntu.content.hub.gschema.xml (+12/-0)
src/com/ubuntu/content/service/hook.cpp (+1/-1)
src/com/ubuntu/content/type.cpp (+6/-0)
To merge this branch: bzr merge lp://staging/~michael-sheldon/content-hub/fix-1383732-rtm
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+246885@code.staging.launchpad.net

Commit message

Add support for EBook content type

Description of the change

Add support for EBook content type

To post a comment you must log in.
Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

Are there any related MPs required for this MP to build/function as expected? Please list.

 * No

Is your branch in sync with latest trunk (e.g. bzr pull lp:trunk -> no changes)

 * Yes (in sync with RTM branch)

Did you perform an exploratory manual test run of your code change and any related functionality on device or emulator?

 * Yes

Did you successfully run all tests found in your component's Test Plan (https://wiki.ubuntu.com/Process/Merges/TestPlan/content-hub) on device or emulator?

 * Yes

If you changed the UI, was the change specified/approved by design?

 * No change

If you changed UI labels, did you update the pot file?

 * No change

If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?

 * No change

Revision history for this message
Michael Sheldon (michael-sheldon) wrote :

There aren't currently any applications that can receive EBooks, so for testing purposes this click can be used to receive ebooks downloaded from the browser: http://mikeasoft.com/~mike/content-hub/com.ubuntu.developer.ken-vandine.hub-importer_0.2_all.click

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Did you perform an exploratory manual test run of the code change and any related functionality on device or emulator?

 * Yes

Did CI run pass? If not, please explain why.

 * Yes

Have you checked that submitter has accurately filled out the submitter checklist and has taken no shortcut?

 * Yes

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