Review Request 129668: Add Armistice day of WWI to holidays for Serbia

Luigi Toscano luigi.toscano at tiscali.it
Sat Jan 14 21:33:45 GMT 2017


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


Ship it!




Applied on current master, compiled, tuned a bit autotests/testholidayregion.cpp to show the holidays for rs_sr at latin and the holiday correctly shows only from 2012.

- Luigi Toscano


On Gen. 14, 2017, 5:09 p.m., Chusslove Illich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129668/
> -----------------------------------------------------------
> 
> (Updated Gen. 14, 2017, 5:09 p.m.)
> 
> 
> Review request for KDEPIM and John Layt.
> 
> 
> Repository: kholidays
> 
> 
> Description
> -------
> 
> Armistice day of World War I is now a public holiday in Serbia. Added to holiday definitions for all four Serbian variants.
> 
> 
> Diffs
> -----
> 
>   holidays/plan2/holiday_rs_sr 6223111 
>   holidays/plan2/holiday_rs_sr at ijekavian 620dc97 
>   holidays/plan2/holiday_rs_sr at ijekavianlatin f5c9d61 
>   holidays/plan2/holiday_rs_sr at latin e3dd519 
> 
> Diff: https://git.reviewboard.kde.org/r/129668/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Chusslove Illich
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170114/216c0120/attachment.html>


More information about the kde-pim mailing list