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

Add date field to eztv #292

Closed

Conversation

ducalex
Copy link
Contributor

@ducalex ducalex commented Aug 13, 2024

They don't provide a convenient format unfortunately, and my parsing might not be exhaustive.

It's also not very accurate when all you have to go by is "1 year [ago]".

But it's certainly better than nothing and seems to work well enough.

NOTE: This PR is branched off current master and doesn't directly depend on any other PR, but since eztv is currently broken then it's not terribly useful until either #291 or #267 is accepted.

image

They don't provide a convenient format unfortunately, and my parsing might not be exhaustive.

It's also not very accurate when all you have to go by is "1 year ago"

But it's certainly better than nothing
@ducalex ducalex force-pushed the ducalex/add-date-field-eztv branch from 0f48c07 to 68c4e36 Compare August 13, 2024 20:36
@xavier2k6 xavier2k6 mentioned this pull request Aug 13, 2024
@xavier2k6
Copy link
Member

@ducalex Please rebase to initiate CI testing & I'll also then test.

@ducalex ducalex closed this Aug 17, 2024
@ducalex ducalex deleted the ducalex/add-date-field-eztv branch August 17, 2024 17:56
@ducalex
Copy link
Contributor Author

ducalex commented Aug 17, 2024

Done

@ducalex
Copy link
Contributor Author

ducalex commented Aug 17, 2024

Oh great I closed the PR accidentally, I guess I have to recreate it?

@ducalex ducalex mentioned this pull request Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants