WARNING: This server is unstable and will be retired in the next days. If you want to keep this forum available, please request immediately a migration on the Nabble Support forum. Forums that don't receive any migration request will be deleted forever.

 « Return to Thread: Re: Review Request: Do not add an organizer to appointments without attendees

Re: Review Request: Do not add an organizer to appointments without attendees

by Kevin Krammer-4 :: Rate this Message:

| View in Thread


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104368/#review12118
-----------------------------------------------------------



incidenceeditor-ng/incidenceattendee.cpp
<http://git.reviewboard.kde.org/r/104368/#comment9544>

    missing block braces


- Kevin Krammer


On April 2, 2012, 3:48 p.m., Stephan Diestelhorst wrote:

>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104368/
> -----------------------------------------------------------
>
> (Updated April 2, 2012, 3:48 p.m.)
>
>
> Review request for KDEPIM and Sergio Luis Martins.
>
>
> Description
> -------
>
> Do not add an organizer to an appointment if there are no attendees present.  Fixed in those locations where it was actually triggered for an appointment that did not have any attendees.  Makes the resulting appointmes editale in Google Calendar and on Android phones.
>
>
> Diffs
> -----
>
>   incidenceeditor-ng/incidenceattendee.cpp 00a86e05507edc86999671eca4f94bbc5d1d07ec
>   incidenceeditor-ng/incidencedefaults.cpp 36f67c9b824b38ea50058c9a7ea2d0879a972178
>
> Diff: http://git.reviewboard.kde.org/r/104368/diff/
>
>
> Testing
> -------
>
> Created several appointments to check that they become editable.  Modified appointments on both phone and KOrganizer and checked they would update on the other respective end.
>
>
> Thanks,
>
> Stephan Diestelhorst
>
>

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

 « Return to Thread: Re: Review Request: Do not add an organizer to appointments without attendees