Discuss recent changes to 'Thanksgiving' holidays in `kr` about definitions HOT 5 OPEN

Hello @ppeble, I'm a native Korean speaker.
First of all, Thank you for your active reaction. 🙂

Here are my answers to the questions above.

1. Can you help us understand the difference in the holiday names as they stand today?

   추석 = Chuseok (More appropriate translation) = Korean Thanksgiving Day (Easy to understand in English)
   But, Google translated from 추석 into Thansgiving. So, I suggested that It is more appropriate for Thanksgiving to be translated as Chuseok to Google Translator.

2. Can you help us understand if our current definitions are correct?

   설날 연휴 (The day before New Year's Day)
   https://github.com/holidays/definitions/blob/master/kr.yaml#L77
   I think 30th day of Dec. should be changed to the last day of last year.

   On the lunar calendar, 2015-12-29 is the last day of year.
   On the lunar calendar, 2016-12-30 is the last day of year.
   The last day of the year in lunar calendar shifting from year to year.

   And If it is now,

   • Now (Wrong)
     설날 연휴 (The day before New Year's Day): lunar_to_solar(2017, 12, 30, :kr)
     설날 (on the Day of New Year's Day): lunar_to_solar(2017, 1, 1, :kr)
     설날 연휴 (The day after New Year's Day): lunar_to_solar(2017, 1, 2, :kr)

   • Correct
     설날 연휴 (The day before New Year's Day): lunar_to_solar(2016, 12, 30, :kr)
     설날 (on the Day of New Year's Day): lunar_to_solar(2017, 1, 1, :kr)
     설날 연휴 (The day after New Year's Day): lunar_to_solar(2017, 1, 2, :kr)

   I hope other native Korean speakers except me will make a confirmation.

3. Can you help us understand why we need to repeat 'Chuseok Holidays' on the 14th and 16th but not the 15th, as defined?

   As the day before and the day after are also part of the holiday.
   To be more specific,
   14th day of August in lunar calendar: The day before Chuseok
   15th day of August in lunar calendar: on the Day of Chuseok
   16th day of August in lunar calendar: The day after Chuseok

   연휴 is a word collectively referring to these words(The day before Chuseok, The day after Chuseok).
   And, I think that would be better to change descriptions more specific.

   Day	Before	After
   The day before New Year's Day	설날 연휴	설날 전날
   The day after New Year's Day	설날 연휴	설날 다음날
   The day before Chuseok	추석 연휴	추석 전날
   The day after Chuseok	추석 연휴	추석 다음날

I am always thankful for your efforts and contributions. 🙂

from definitions.

@swcheon Thank you so much! I'm sorry that I didn't get back to this right away, just more baby stuff came up and I ran out of free time. 😄

I will take a look at this in the coming days and formulate a plan. Thank you again!

from definitions.

@ppeble I agree with your suggestions. 전날 and 다음날 are more correct words. But I do not use that word in the real world. I think 추석 연휴 is not so bad. How about using 추석 연휴 as it is?

from definitions.

@EunJung-Seo I think that is also good.

from definitions.

if our current definitions are correct?

No, Current logic does not care Substitute holiday.
After 2013, 설날 연휴 and 추석 연휴 is applied Substitute holiday.

as described in 관공서의 공휴일에 관한 규정 일부개정령안:
If the holiday(or Sunday) falls on a New Year(설날)'s Day or a Chuseok(추석) holiday, the first business day following that day shall be a holiday. If the Children's Day(어린이날) falls on a Saturday or another holiday (Sunday, Buddha's birthday(석가탄신일), etc.), the first business day following that day shall be a holiday.

After this rule, correction of this year's 설날 holiday like this:

• 2017 27 Jan(Fri) - 설날 연휴
• 2017 28 Jan(Sat) - 설날
• 2017 29 Jan(Sun) - not holiday
• 2017 30 Jan(Mon) - 대체 휴일

ref here: https://ko.wikipedia.org/wiki/%EB%8C%80%EC%B2%B4%ED%9C%B4%EC%9D%BC%EC%A0%9C%EB%8F%84

from definitions.