-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
isToday(new UTCDate()) => false #3730
Comments
There might be a need to audit all A fix for // currently:
return isSameDay(date, Date.now());
// fix:
return isSameDay(date, constructFrom(date, Date.now())); It would be a good idea to devise a testing strategy for this type of issue between the two projects. |
@fturmel, you're right about that. Thank you for pointing the way. I will review it and ship a fix. |
Fixed via b1a5eb5 |
Shipped with |
There is an issue with the isToday function
The problem
💻 Code demonstrating the problem
Demo: https://stackblitz.com/edit/stackblitz-starters-bdluhl?file=src%2Fmain.ts
🙁 Actual behavior
new UTCDate()
is not today🙂 Expected behavior
new UTCDate()
is todayDebug information
The text was updated successfully, but these errors were encountered: