[Marble-devel] Review Request: Foursquare authentication

[Commit Hook]

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


This review has been submitted with commit de712ba72b0d2a1870eb094060b034c75136f3ca by Utku Aydın to branch sok-2012-plasma-active.

- Commit Hook


On Oct. 30, 2012, 8:11 p.m., Utku Aydın wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107011/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2012, 8:11 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> With this patch, Foursquare plugin users now can allow Marble to make requests to Foursquare on behalf of them. Currently works only on Marble Touch, and has minor problems, e.g. the login page is not scrollable.
> 
> The patch also fixes the problem with the venue details sidebar, widens the sidebar if the detail texts are longer than it. It also adds the missing declarativeModel method to DeclarativeDataPlugin class.
> 
> 
> Diffs
> -----
> 
>   src/plugins/declarative/DeclarativeDataPlugin.cpp 73406b8 
>   src/plugins/render/foursquare/FoursquarePlugin.h fc75f06 
>   src/plugins/render/foursquare/FoursquarePlugin.cpp ae8210c 
>   src/qt-components/marble-touch/activities/Explore.qml 85d4344 
> 
> Diff: http://git.reviewboard.kde.org/r/107011/diff/
> 
> 
> Testing
> -------
> 
> Printed the API key using qDebug(), got a string that looks like an API key.
> 
> 
> Thanks,
> 
> Utku Aydın
> 
>

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