Merge lp://staging/~macslow/unity-notifications/combo-button-support into lp://staging/unity-notifications

Proposed by Mirco Müller
Status: Merged
Approved by: Michał Sawicz
Approved revision: 210
Merged at revision: 211
Proposed branch: lp://staging/~macslow/unity-notifications/combo-button-support
Merge into: lp://staging/unity-notifications
Diff against target: 172 lines (+53/-21) (has conflicts)
4 files modified
debian/changelog (+11/-0)
examples/sd-example-incoming-call-canceled.py (+17/-10)
examples/sd-example-incoming-call.py (+24/-10)
src/NotificationServer.cpp (+1/-1)
Text conflict in debian/changelog
To merge this branch: bzr merge lp://staging/~macslow/unity-notifications/combo-button-support
Reviewer Review Type Date Requested Status
Michał Sawicz Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+221506@code.staging.launchpad.net

Commit message

Updated calling snap-decision examples to show off new combo-button support in notifications frontend (unity8). Updated the incoming-call example to use 7 actions to be inline with design-requirements.

Description of the change

Updated calling snap-decision examples to show off new combo-button support in notifications frontend (unity8). Updated the incoming-call example to use 7 actions to be inline with design-requirements.

- Are there any related MPs required for this MP to build/function as expected?
No

- Did you perform an exploratory manual test run of your code change and any related functionality?
Yes, but only on desktop not yet on the device/phone since cross-compiling currently doesn't work for me (Malta-sprint)

- Did you make sure that your branch does not contain spurious tags?
Yes

- If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Yes

- If you changed the UI, has there been a design review?
not applicable

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

This looks fine, we just need a changelog bump so that autopilot tests in the unity8 counterpart can depend on this change.

review: Needs Fixing
210. By Mirco Müller

Bumped version to indicate the updated maximum number of supported actions.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Yup.

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: