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

[HiDive] Remove unreleased episodes from list #685

Merged
merged 4 commits into from
May 21, 2024
Merged

Conversation

Denoder
Copy link
Contributor

@Denoder Denoder commented May 9, 2024

When doing a range download/single selection of an anime that's currently airing episodes that haven't released will still get downloaded at 0 bytes. This will skip them if they have a date range in them, rather than an appropriate title.

When doing a range download/single selection of an anime that's currently airing episodes that haven't released will still get downloaded at 0 bytes. This will skip them if they have a date range in them, rather than an appropriate title.
@AnimeDL
Copy link
Contributor

AnimeDL commented May 18, 2024

I apologize for taking so long for getting to review this, I was quite busy on another one of my projects, the change LGTM and appears to work. I am curious on your thoughts though, should it still list it in the output, and just ignore those episodes if requested instead (could be done by just not pushing those episodes to the array)? Incase anybody wants to know when the next episodes come out

Denoder added 2 commits May 19, 2024 00:06
Account for when the episode title is added, but still not released, when it's not available there's a 10 second clip.
Allow listing to be visible but now allow it in the episodes list if it's not available
@Denoder
Copy link
Contributor Author

Denoder commented May 18, 2024

Yea i think that'll work

@AnimeDL
Copy link
Contributor

AnimeDL commented May 21, 2024

I could be crazy, but I think that this has broken downloading on hidive now. Season ID 22309, can't seem to download from it. No episodes end up getting pushed to the array I think

@Denoder
Copy link
Contributor Author

Denoder commented May 21, 2024

im not having this issue:

@Denoder
Copy link
Contributor Author

Denoder commented May 21, 2024

Okay i saw my own mistake, try now.

@AnimeDL
Copy link
Contributor

AnimeDL commented May 21, 2024

Yup, that fixed it. I think it's good to merge now

@AnimeDL AnimeDL merged commit 38f849f into anidl:master May 21, 2024
2 checks passed
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