Skip to content
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

Small date mapping fix #2584

Merged
merged 3 commits into from Nov 11, 2023
Merged

Small date mapping fix #2584

merged 3 commits into from Nov 11, 2023

Conversation

certuna
Copy link
Contributor

@certuna certuna commented Nov 4, 2023

fallback in the case there's no Date tagged but Original Date or Release Date are tagged

fallback in the case there's no Date tagged but Original Date or Release Date are tagged
Copy link

github-actions bot commented Nov 4, 2023

Download the artifacts for this pull request:

@deluan
Copy link
Member

deluan commented Nov 11, 2023

Nice catch! I just added tests to it.

@deluan deluan merged commit 40841ab into navidrome:master Nov 11, 2023
5 checks passed
@metalheim
Copy link
Contributor

metalheim commented Nov 15, 2023

Can confirm this fixes the issue for me.
For anyone else who was using the develop branch before this fix: The fix needs a full rescan to pick up the missing date, or alternatively (what i did) you can touch the files and they will be updated on next scheduled (quick) scan

Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants