[Kde-pim] Review Request 119434: Per attendee customization of email [2] - DialogInterception for IncienceChanger

[Sandro Knauß]

> On Juli 24, 2014, 5:26 vorm., Laurent Montel wrote:
> > akonadi/calendar/incidencechanger_p.h, line 118
> > <https://git.reviewboard.kde.org/r/119434/diff/1/?file=292205#file292205line118>
> >
> >     don't put code in .h file please

Well there is more code in incidencechanger_p.h maybe it should be moved out there completly in another patch. But until than I think it is the right possition.


- Sandro


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


On Juli 23, 2014, 8:50 nachm., Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119434/
> -----------------------------------------------------------
> 
> (Updated Juli 23, 2014, 8:50 nachm.)
> 
> 
> Review request for KDEPIM-Libraries.
> 
> 
> Repository: kdepimlibs
> 
> 
> Description
> -------
> 
> To be able to customize the emails per attendee from an iTip event. The dialogs opened by IncidenceChange has to be interceptable. You can see the both changeset in the dev/individual_mail2 branch.
> 
> 
> Diffs
> -----
> 
>   akonadi/calendar/incidencechanger_p.h d1c0e80531d8daef5c393c10fd9d5e322cb88293 
>   akonadi/calendar/itiphandler.h 56aaedaca2a33b74f2206976313d645d7b50d4b2 
>   akonadi/calendar/itiphandler.cpp 03a3adb9e4201c835b7be7ceade4c57285df13a8 
>   akonadi/calendar/itiphandler_p.h a0bb98e1380fc91c4bab3931b88a0f8212007f9c 
>   akonadi/calendar/itiphandler_p.cpp 4d024bc597ce77882424e522f06b46e376e1c3d5 
>   akonadi/calendar/itiphandlerhelper_p.h 466bd8c0010dcdf1e8e03469e99942e10af50ac4 
>   akonadi/calendar/itiphandlerhelper_p.cpp 1b6d03f03f54c955f8a704e9121d7745b1cfbf8e 
>   akonadi/calendar/tests/CMakeLists.txt 382e3ec81e189085e4fbf9f34a3be9e6511e52f7 
>   akonadi/calendar/incidencechanger.cpp 0059f2aa37c7af0e0101b2a0540e64aaf8d83258 
> 
> Diff: https://git.reviewboard.kde.org/r/119434/diff/
> 
> 
> Testing
> -------
> 
> run multiple times kincidenceeditor and korganizer to test and run itiphelpertest and mailclienttest.
> 
> 
> 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/