Merge lp:~dpm/reminders-app/drop-qtwebkit into lp:reminders-app
Proposed by
David Planella
Status: | Merged |
---|---|
Approved by: | Michael Zanetti |
Approved revision: | 122 |
Merged at revision: | 121 |
Proposed branch: | lp:~dpm/reminders-app/drop-qtwebkit |
Merge into: | lp:reminders-app |
Diff against target: |
25 lines (+0/-5) 2 files modified
src/app/qml/ui/NotePage.qml (+0/-3) src/app/qml/ui/NoteView.qml (+0/-2) |
To merge this branch: | bzr merge lp:~dpm/reminders-app/drop-qtwebkit |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Michael Zanetti (community) | Approve | ||
Review via email: mp+218560@code.launchpad.net |
Commit message
Drop unused imports to QtWebkit
Description of the change
When creating a click package, the click reviewers tools complain on:
Found files that use unsupported QtWebKit (should use UbuntuWebview or Oxide instead): qml/ui/NoteView.qml ,qml/ui/
I've initially removed the (unused) imports on NotePage.qml and NoteView.qml.
To post a comment you must log in.
FAILED: Continuous integration, rev:122 /code.launchpad .net/~dpm/ reminders- app/drop- qtwebkit/ +merge/ 218560/ +edit-commit- message
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http:// 91.189. 93.70:8080/ job/reminders- app-ci/ 297/ 91.189. 93.70:8080/ job/generic- mediumtests- trusty/ 2476 91.189. 93.70:8080/ job/reminders- app-saucy- amd64-ci/ 297 91.189. 93.70:8080/ job/reminders- app-trusty- amd64-ci/ 297
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/reminders- app-ci/ 297/rebuild
http://