<table><tr><td style="">nhiga added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28874">View Revision</a></tr></table><br /><div><div><p>Thank you for your attempt to add Taiwanese holidays to KDE. However, I am afraid that this patch has to be reviewed again.</p>

<p>The dates for some holidays (including, but not limited to, Spring Festival (also known as "Lunar New Year") and the Mid-Autumn Festival) are different every year because some holidays are determined based on the Lunar Calendar (農曆). That means one has to specify the exact dates for each year, just like the <tt style="background: #ebebeb; font-size: 13px;">holiday_hk_*</tt> files.<br />
If you look at the table in the <a href="https://en.wikipedia.org/wiki/Public_holidays_in_Taiwan" class="remarkup-link" target="_blank" rel="noreferrer">English Wikipedia entry for Public holidays in Taiwan</a> carefully, you will notice that some holidays follow Chinese calendar instead of the Gregorian calendar.</p>

<p>Therefore, we should try to refer to the relevant laws and regulations, and use government-published documents as sources for accuracy. In this case, we should refer to the law <a href="https://law.moj.gov.tw/LawClass/LawAll.aspx?pcode=D0020033)" class="remarkup-link" target="_blank" rel="noreferrer">"紀念日及節日實施辦法" (only available in Chinese)</a>. In addition, the Ministry of the Interior, R. O. C. (Taiwan) publishes a document listing the exact dates of public holidays each year ("public" here means "放假", carrying the meaning of being a day off). The document for 2020 can be found <a href="https://www.moi.gov.tw/files/news_file/109%E5%B9%B4%E6%87%89%E6%94%BE%E5%81%87%E4%B9%8B%E7%B4%80%E5%BF%B5%E6%97%A5%E5%8F%8A%E7%AF%80%E6%97%A5(odt).odt" class="remarkup-link" target="_blank" rel="noreferrer">here</a> (Note that the ROC era is used, so Year 109 means 2020).</p>

<p>The public holidays as well as the official commemorative holidays are already complicated, so I think for the time being it would be better to exclude non-official holidays such as Halloween or Christmas.</p>

<p>Thank you for your attention.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D28874#inline-166775">View Inline</a><span style="color: #4b4d51; font-weight: bold;">holiday_tw_zh:16</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">country     "TW"
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">language    "tw_zh"
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">:name        "optional - defaults to country name"
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Shouldn't this line be <tt style="background: #ebebeb; font-size: 13px;">zh</tt> (or <tt style="background: #ebebeb; font-size: 13px;">zh_TW</tt>) instead? It looks like the format for the "language" metadata entry has been <tt style="background: #ebebeb; font-size: 13px;"><language></tt> or <tt style="background: #ebebeb; font-size: 13px;"><language>_<region></tt>.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R175 KHolidays</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28874">https://phabricator.kde.org/D28874</a></div></div><br /><div><strong>To: </strong>shrapnel, VDG, Zren, winterz<br /><strong>Cc: </strong>nhiga, ngraham, winterz, kde-frameworks-devel, LeGast00n, cblack, michaelh, bruns<br /></div>