Review Request 117689: Netherlands: Koninginnedag is now Koningsdag

[Albert Astals Cid]

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



Do we know how to fix this? Also worst case scenario better to have "future dates" with the holiday being correct even if that means "old dates" also show the holiday than nothing, no?

- Albert Astals Cid


On April 28, 2014, 1:21 p.m., Thijs Kinkhorst wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117689/
> -----------------------------------------------------------
> 
> (Updated April 28, 2014, 1:21 p.m.)
> 
> 
> Review request for KDEPIM-Libraries and John Layt.
> 
> 
> Bugs: 331219
>     http://bugs.kde.org/show_bug.cgi?id=331219
> 
> 
> Repository: kdepimlibs
> 
> 
> Description
> -------
> 
> Starting with 2014, Queen's Day will be King's Day and celebrated
> on the 27th of April, unless that's a Sunday, then it will be a
> Saturday.
> 
> http://nl.wikipedia.org/wiki/Koningsdag_%28Nederland%29#Datum
> http://en.wikipedia.org/wiki/Koningsdag#Willem-Alexander
> 
> 
> Diffs
> -----
> 
>   kholidays/holidays/plan1/holiday_nl_nl 3027170 
>   kholidays/holidays/plan2/holiday_nl_nl fe9f263 
>   kholidays/tests/holiday_xx_xx_Old 95e1e24 
> 
> Diff: https://git.reviewboard.kde.org/r/117689/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Thijs Kinkhorst
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170204/2eb44626/attachment.html>