-
-
Notifications
You must be signed in to change notification settings - Fork 423
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
fix: Properly parse date strings #4924
fix: Properly parse date strings #4924
Conversation
today when i created a branch to work on the instance button issue, i merged this and app is crashing with same datetimeparse errors, but the current alpha with new dates is working fine! |
What's the date format displayed in the error message? Update: I made a few more changes, there should be no issues now. |
9592c49
to
69fbb99
Compare
yes, it's perfect now. |
We currently still only use the date without time. Wouldn't it make more sense to use |
Yeah, it would. |
223bd87
to
1c8e5c5
Compare
app/src/main/java/com/github/libretube/db/obj/WatchHistoryItem.kt
Outdated
Show resolved
Hide resolved
1c8e5c5
to
36ae0e8
Compare
Properly parse the new date format, which contains a time offset.