[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    [Kde-pim] Re:
From:       Thomas McGuire <mcguire () kde ! org>
Date:       2011-04-04 22:48:52
Message-ID: 201104042348.57898.mcguire () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


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

["signature.asc" (application/pgp-signature)]

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/

[prev in list] [next in list] [prev in thread] [next in thread]