Review Request: Small fixes in AbstractAccountParametersWidget to make rakia advanced options work properly

Daniele Elmo Domenichelli daniele.domenichelli at gmail.com
Mon Jun 11 14:20:44 UTC 2012


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

Ship it!


Ship It!

- Daniele Elmo Domenichelli


On June 11, 2012, 12:51 p.m., George Kiagiadakis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105220/
> -----------------------------------------------------------
> 
> (Updated June 11, 2012, 12:51 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Some fixes to make the rakia advanced options widget work properly...
> 
> commit 32c6caf33392bbafd0ad10e1e4d0ed45ac6645d5
> Author: George Kiagiadakis <george.kiagiadakis at collabora.com>
> Date:   Mon Jun 11 13:55:44 2012 +0300
> 
>     AbstractAccountParametersWidget: Reset parameters with disabled widgets to their default values when submitting changes
> 
> commit 2c2f47bdad9e55a45da2fae15e3f33c3f199caa8
> Author: George Kiagiadakis <george.kiagiadakis at collabora.com>
> Date:   Mon Jun 11 13:41:33 2012 +0300
> 
>     AbstractAccountParametersWidget: Remove unused variable
> 
> commit 42e919326ee0e3ecb2f0815f42bc28fb7925bc78
> Author: George Kiagiadakis <george.kiagiadakis at collabora.com>
> Date:   Mon Jun 11 13:30:56 2012 +0300
> 
>     AbstractAccountParametersWidget: Map the "currentText" property of QComboBox if the parameter requires a string type
> 
> 
> Diffs
> -----
> 
>   src/KCMTelepathyAccounts/abstract-account-parameters-widget.cpp b61e836b77809819a0f6f24f9b9b1bfe2a3af6a6 
> 
> Diff: http://git.reviewboard.kde.org/r/105220/diff/
> 
> 
> Testing
> -------
> 
> Tested setting/unsetting some properties and verifying with qdbus o.f.T.AccountManager ... that the stored parameters are correct (unlike before).
> 
> 
> Thanks,
> 
> George Kiagiadakis
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120611/2860de00/attachment.html>


More information about the KDE-Telepathy mailing list