[Kde-pim] Re: Free-busy integration in KDEPIM - Osnabrück outcome

[Thomas McGuire]

Hi,

On Monday, April 04, 2011 08:33:29 PM Grégory Oestreicher wrote:
> Le dimanche 03 avril 2011 21:34:01, Thomas McGuire a écrit :
> > For the kdepimlibs parts, they look quite ok, maybe they should be in the
> > kcal/ subdir of akonadi though? (yeah we missed that for the transport
> > resource...)
> 
> Done. Everything will be pushed into the clones later tonight.

Thanks.

> > Can lastCacheUpdate() not use a KDateTime instead of a QString?
> 
> I was so much focused on the DAV groupware implementation that I totally
> forgot this as it doesn't make sense for this resource. I can make this a
> KDateTime, but it will have to be converted to a QString to send it over D-
> Bus. In that case it's FreeBusyProviderBasePrivate that will to the
> conversion, but the receiver will have to use a QString. Unless there's
> already a D-Bus type for KDateTime.

Sounds good.

> > Maybe instead of using a boolean to indicate an error, have a QString for
> > the user-visible error message somewhere, that KOrganizer could display?
> 
> Why not, but only the latest message submitted over D-Bus will be kept in
> the case that two resources answer with a failure. I assume that you are
> only talking about the freebusyRetrieved() call.

Yes, I was talking about the freebusyRetrieved() call. The fact that only the 
latest message will be kept in memory is an implementation detail of the 
free/busy manager, and not an API issue, right?

Regards,
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20110404/cf516888/attachment.sig>
-------------- next part --------------
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/