[Marble-devel] Review Request 114270: Give Feedback Whether Owncloud Login Credentials are Correct in Marble

Benjamin Kaiser benjaminjkaiser at gmail.com
Sun Dec 8 10:59:15 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114270/
-----------------------------------------------------------

(Updated Dec. 8, 2013, 10:59 a.m.)


Status
------

This change has been marked as submitted.


Review request for Marble and Dennis Nienhüser.


Repository: marble


Description
-------

Give Feedback Whether Owncloud Login Credentials are Correct in Marble.

This has been implemented for both the marble-qt and marble-kde applications settings. Note however that I had to add UI code to both their classes, because the UI building code was not inside the ControlView class. The logic behind the login credential checking was implemented in the correct place (OwncloudSyncBackend).

GCI task: http://www.google-melange.com/gci/task/view/google/gci2013/5900404730626048


Diffs
-----

  src/apps/marble-kde/marble_part.h ddb91ea 
  src/apps/marble-kde/marble_part.cpp 082ad5e 
  src/apps/marble-qt/QtMainWindow.cpp a713272 
  src/lib/marble/MarbleCloudSyncSettingsWidget.ui 9257dac 
  src/lib/marble/QtMarbleConfigDialog.h 95be36d 
  src/lib/marble/QtMarbleConfigDialog.cpp b8a85c0 
  src/lib/marble/cloudsync/CloudSyncManager.h 7955bc8 
  src/lib/marble/cloudsync/CloudSyncManager.cpp caa9536 
  src/lib/marble/cloudsync/OwncloudSyncBackend.h 715f60c 
  src/lib/marble/cloudsync/OwncloudSyncBackend.cpp 8af8182 

Diff: http://git.reviewboard.kde.org/r/114270/diff/


Testing
-------

Run both the `marble` (marble-kde) and `marble-qt` applications. 


Thanks,

Benjamin Kaiser

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20131208/820a94c2/attachment.html>


More information about the Marble-devel mailing list