Merge lp:~rpadovani/reminders-app/multipleAccounts into lp:reminders-app
Proposed by
Riccardo Padovani
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Michael Zanetti | ||||
Approved revision: | 267 | ||||
Merged at revision: | 291 | ||||
Proposed branch: | lp:~rpadovani/reminders-app/multipleAccounts | ||||
Merge into: | lp:reminders-app | ||||
Diff against target: |
249 lines (+79/-36) 9 files modified
3rdParty/libthrift/transport/TSSLSocket.cpp (+5/-3) src/app/qml/reminders.qml (+8/-1) src/app/qml/ui/AccountSelectorPage.qml (+44/-22) src/app/qml/ui/NotebooksPage.qml (+1/-1) src/app/qml/ui/NotesPage.qml (+1/-1) src/app/qml/ui/RemindersPage.qml (+1/-1) src/plugin/Evernote/evernoteconnection.cpp (+3/-0) src/plugin/Evernote/notesstore.cpp (+14/-7) src/plugin/Evernote/notesstore.h (+2/-0) |
||||
To merge this branch: | bzr merge lp:~rpadovani/reminders-app/multipleAccounts | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Zanetti (community) | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
David Planella | Needs Fixing | ||
Review via email: mp+237682@code.launchpad.net |
Commit message
Fixed change accounts page
Description of the change
Implemented new change accounts page as discussed during lasts meetings.
When you change account you don't see account name due https:/
For the same reason, autologin doesn't work
To post a comment you must log in.
PASSED: Continuous integration, rev:259 91.189. 93.70:8080/ job/reminders- app-ci/ 543/ 91.189. 93.70:8080/ job/generic- mediumtests- utopic- python3/ 779 91.189. 93.70:8080/ job/generic- mediumtests- utopic- python3/ 779/artifact/ work/output/ *zip*/output. zip 91.189. 93.70:8080/ job/reminders- app-utopic- amd64-ci/ 239
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/reminders- app-ci/ 543/rebuild
http://