Review Request 116558: Add KAccounts support to the dav resource

Àlex Fiestas afiestas at kde.org
Thu Mar 6 21:24:07 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116558/#review52299
-----------------------------------------------------------


For the release time, I would like to give some feedback on how this patch can break things (given that we are already way pass freeze).

The first thing this patch does is adding two extra settings called "AccountId" and "AccountServices". By default these settings are empty and unless initialized to some value this patch will do nothing.

In the case that those settings are set, then we create "urls". This url have a type build in that indicates mostly from where the password is fetched. The url created by this patch have the type "accounts" and this patch will affect only the urls create by this patch. To put it short this patch does not affect the workflow of the code for urls (dav servers) that are not configured by this patch.

So, if we imagine that my patch is bugged to the point where it makes the resource crash, it won't affect other instances of the resource.

- Àlex Fiestas


On March 6, 2014, 7:55 p.m., Àlex Fiestas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116558/
> -----------------------------------------------------------
> 
> (Updated March 6, 2014, 7:55 p.m.)
> 
> 
> Review request for KDEPIM, Release Team and Grégory Oestreicher.
> 
> 
> Repository: kdepim-runtime
> 
> 
> Description
> -------
> 
> If compiled with KAccounts support and configure to use it, the resource will fetch credentials and host using libaccounts in order to build the urls.
> 
> Support for "editing" is not done, that will be done in 4.14 since it requires better kaccounts-integration.
> 
> To the release team, I would like to add this change to 4.13, it won't do anything except compiled with libaccounts and even so only if the resource is configured to use it.
> 
> 
> Diffs
> -----
> 
>   resources/dav/resource/CMakeLists.txt 14675bc 
>   resources/dav/resource/davgroupwareresource.desktop 2342a01 
>   resources/dav/resource/davgroupwareresource.kcfg 2d25371 
>   resources/dav/resource/settings.h e264866 
>   resources/dav/resource/settings.cpp 12f84cc 
> 
> Diff: https://git.reviewboard.kde.org/r/116558/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Àlex Fiestas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20140306/a1129fb5/attachment.html>


More information about the release-team mailing list