[Kde-pim] Review Request 119518: WIP: Add support for tags to the Kolab resource

[Kevin Krammer]

> On Juli 28, 2014, 2:12 nachm., Christian Mollekopf wrote:
> > resources/kolab/kolabhelpers.cpp, line 325
> > <https://git.reviewboard.kde.org/r/119518/diff/1/?file=293855#file293855line325>
> >
> >     I'm not sure we actually need to store the kolab objects locally (and therefore sync the collection). We should be able to directly create the tags from the kolab object without storing the (useless) kolab object locally.

As far as I could tell, the collection is mainly needed to store a UidNextAttribute. At least that is what the similar IMAP tasks do (item add, item remove, item change)


> On Juli 28, 2014, 2:12 nachm., Christian Mollekopf wrote:
> > resources/kolab/kolabaddtagtask.cpp, line 65
> > <https://git.reviewboard.kde.org/r/119518/diff/1/?file=293850#file293850line65>
> >
> >     At least not yet. Perhaps implement it as custom task that is triggered by a collection sync?

Ok.


- Kevin


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


On Juli 28, 2014, 1:23 nachm., Kevin Krammer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119518/
> -----------------------------------------------------------
> 
> (Updated Juli 28, 2014, 1:23 nachm.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Repository: kdepim-runtime
> 
> 
> Description
> -------
> 
> Converts Akonadi::Tag to Kolab::Relation and stores the in a Configuration mailbox/collection.
> 
> 
> Diffs
> -----
> 
>   resources/kolab/CMakeLists.txt 0e8e237a6b9f3d1f77154a0625981ade770a23da 
>   resources/kolab/kolabaddtagtask.h PRE-CREATION 
>   resources/kolab/kolabaddtagtask.cpp PRE-CREATION 
>   resources/kolab/kolabchangeitemstagstask.h PRE-CREATION 
>   resources/kolab/kolabchangeitemstagstask.cpp PRE-CREATION 
>   resources/kolab/kolabchangetagtask.h PRE-CREATION 
>   resources/kolab/kolabchangetagtask.cpp PRE-CREATION 
>   resources/kolab/kolabhelpers.cpp 084bcb537aa7d1eb3a2de4d94413aea04256cf98 
>   resources/kolab/kolabrelationresourcetask.h PRE-CREATION 
>   resources/kolab/kolabrelationresourcetask.cpp PRE-CREATION 
>   resources/kolab/kolabremovetagtask.h PRE-CREATION 
>   resources/kolab/kolabremovetagtask.cpp PRE-CREATION 
>   resources/kolab/kolabresource.h 38889ad9e77467c80353fc61ba808ec107fccd32 
>   resources/kolab/kolabresource.cpp a5e051a56b5579e51e461e876b337b5a7306405b 
> 
> Diff: https://git.reviewboard.kde.org/r/119518/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kevin Krammer
> 
>

_______________________________________________
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/