[Kde-pim] Review Request 123521: kolab changes: calendersupport

[Laurent Montel]

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



calendarsupport/kcalprefs.h (line 101)
<https://git.reviewboard.kde.org/r/123521/#comment54339>

    could you put it in a separate file please ?
    It's better to have one class by file.
    Thanks



calendarsupport/kcalprefs.cpp (line 42)
<https://git.reviewboard.kde.org/r/123521/#comment54340>

    use calendarsupport_debug.cpp



calendarsupport/kcalprefs.cpp (line 358)
<https://git.reviewboard.kde.org/r/123521/#comment54341>

    use new connect api



calendarsupport/kcalprefs.cpp (line 390)
<https://git.reviewboard.kde.org/r/123521/#comment54342>

    new connect api



calendarsupport/kcalprefs.cpp (line 396)
<https://git.reviewboard.kde.org/r/123521/#comment54343>

    qCWarning(...)


- Laurent Montel


On avr. 27, 2015, 8:59 matin, Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123521/
> -----------------------------------------------------------
> 
> (Updated avr. 27, 2015, 8:59 matin)
> 
> 
> Review request for KDEPIM and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> CalendarSupport: show displayName if available for collections
> 
> 
> CalenderSupport: handles now also KCalCore::Incidence::Ptr [1/2]
> 
> need for CalendarViews: Get rid of Akonadi::Item in Agendaview [2/2]
> 
> CalendarSupport/KCalPrefs: Store category colors in tags.
> 
> existance of the BlockAlarmAttribute != blocking alarms
> 
> Check if all alarms are blocked, to activate the checkeckbox "block
> reminders locally". Background is, that we have to descide, if an user
> has set this value by his own or not.
> 
> KOLAB: #3731
> 
> 
> Diffs
> -----
> 
>   calendarsupport/collectiongeneralpage.cpp 45cf9e40c477fa5e12b8836c024feb7d40e8f53a 
>   calendarsupport/kcalprefs.h 3788b9ea3d4d82e6abba7dfd672fbc3e71990ec8 
>   calendarsupport/kcalprefs.cpp 56c4433deecdf92ce93b06cf4dea2909cd83ef9f 
>   calendarsupport/utils.h c5aced2857d9171cb977aadf516e39c212316d73 
>   calendarsupport/utils.cpp 4e65a960139c0ecddb1c1e13d1930de3f441c6d9 
> 
> Diff: https://git.reviewboard.kde.org/r/123521/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sandro Knauß
> 
>

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