Merge lp:~boiko/dialer-app/do_not_call_directly into lp:dialer-app
Status: | Merged |
---|---|
Approved by: | Bill Filler |
Approved revision: | 282 |
Merged at revision: | 284 |
Proposed branch: | lp:~boiko/dialer-app/do_not_call_directly |
Merge into: | lp:dialer-app |
Diff against target: |
34 lines (+3/-10) 2 files modified
src/qml/HistoryPage/HistoryDelegate.qml (+2/-5) src/qml/HistoryPage/HistoryDetailsPage.qml (+1/-5) |
To merge this branch: | bzr merge lp:~boiko/dialer-app/do_not_call_directly |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Bill Filler (community) | Approve | ||
PS Jenkins bot | continuous-integration | Needs Fixing | |
Review via email: mp+235039@code.launchpad.net |
Commit message
Do not call the phone numbers directly when clicking the items in call log.
Clicking them should only populate dialpad.
Description of the change
Do not call the phone numbers directly when clicking the items in call log.
Clicking them should only populate dialpad.
== 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?
Yes
If you changed UI labels, did you update the pot file?
N/A
If you changed the packaging (debian), did you add a core-dev as a reviewer to this MP?
N/A
FAILED: Continuous integration, rev:281 jenkins. qa.ubuntu. com/job/ dialer- app-ci/ 456/ jenkins. qa.ubuntu. com/job/ dialer- app-utopic- i386-ci/ 227 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- utopic- touch/4918 jenkins. qa.ubuntu. com/job/ generic- mediumtests- utopic/ 3592 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- mako/4672 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/6170 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/6170/ artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 13346 jenkins. qa.ubuntu. com/job/ autopilot- testrunner- otto-utopic/ 2980 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- amd64/3901 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- amd64/3901/ artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/dialer- app-ci/ 456/rebuild
http://