Merge lp://staging/~tiagosh/telepathy-ofono/fix-voicemail-indicator into lp://staging/telepathy-ofono
Proposed by
Tiago Salem Herrmann
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Gustavo Pichorim Boiko | ||||
| Approved revision: | 89 | ||||
| Merged at revision: | 89 | ||||
| Proposed branch: | lp://staging/~tiagosh/telepathy-ofono/fix-voicemail-indicator | ||||
| Merge into: | lp://staging/telepathy-ofono | ||||
| Diff against target: |
73 lines (+13/-0) 3 files modified
connection.cpp (+1/-0) voicemailiface.cpp (+6/-0) voicemailiface.h (+6/-0) |
||||
| To merge this branch: | bzr merge lp://staging/~tiagosh/telepathy-ofono/fix-voicemail-indicator | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Gustavo Pichorim Boiko (community) | Approve | ||
| PS Jenkins bot | continuous-integration | Needs Fixing | |
|
Review via email:
|
|||
Commit message
Add VoicemailNumber
Description of the change
Add VoicemailNumber
--Checklist--
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
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:/
Yes
If you changed the UI, was the change specified/approved by design?
N/A
If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?
N/A
To post a comment you must log in.
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | # |
review:
Needs Fixing
(continuous-integration)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
FAILED: Continuous integration, rev:89
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://