-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
[bugfix]: switch to UTC for release dates #743
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While this is fine for release date, it is not necessarily fine for things like createdAt/updatedAt (date added), which from Navidrome at least include timezone. It would be more correct to conditionally call local()
for certain locations
In that case, would you prefer the default being left as local and then adding a UTC version? |
Thank you for fixing this! I have entries tagged When this is implemented, will that |
You can test this live on https://feishin.vercel.app, but in my testing it would be |
Just tried it and there's indeed a bug present where it incorrectly states it was released
As someone who has no dev experience, I have no idea what any of this means :P Is it a way for users to fix the above problem? |
I don't recall there ever being an option for it to say |
Checking my Navidrome GUI, it correctly says Just took a look at a different release that is tagged with only
Would it be appropriate if I submit a separate issue for this to be changed / implemented? I suppose it comes down to the concept that displaying no information is better than displaying wrong information; one I would assume most of us would agree with :P |
This commit switches DayJS to use UTC rather then local time when processing absolute dates, this fixes a bug where albums would show as releasing a day before they really did.
Some quick tests I did:
createdAt
anddateAdded
fields are still correct.If you see anything wrong let me know, but I've tested every file that uses this function and they all seem to work correctly.