Calculates Day After Thanksgiving incorrectly for 2019

[AndrewLane]

Looks like the library assumes that the Day After Thanksgiving is the 4th Friday of November based on this code:

moment-holiday/locale/united_states.js

Line 73 in 447e7ff

date: '11/(5,4)',

However, on years like 2019 when the first day of the month of November is Friday, this means that the code will actually calculate the day after Thanksgiving as being before Thanksgiving itself. To repro:

var momentHoliday = require("moment-holiday")
var moment = require('moment');

var newYears2019 = moment('2019 01 01', 'YYYY MM DD');

console.log(momentHoliday(newYears2019).holiday('Thanksgiving').format('YYYY MM DD'));
console.log(momentHoliday(newYears2019).holiday('Day After Thanksgiving').format('YYYY MM DD'));

Expected:

2019 11 28
2019 11 29

Actual:

2019 11 28
2019 11 22

[huangxuewu]

same issue

[AndrewLane]

FYI I added a test that demonstrates the bug here: #34

I hope it's helpful.

[jorgedrr]

I have the same issue

[smarth55]

date: '11/(5,[23])' does the job. I added a custom day after thanksgiving with that date, but I can make a PR with the change.

[AndrewLane]

@smarth55 actually at this point i'm wondering if we should consider this a dead project since I haven't seen commits since 2017.

[joeycozza]

The day of reckoning is upon us!!!

Hopefully no one has important client facing code relying on this! Our little slack bot tool didn't give us our notifications this morning because of this bug, but that doesn't REALLY matter. Good luck all moment-holiday users!