-
Notifications
You must be signed in to change notification settings - Fork 4
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
Account for TPB chapter numbers in ReadComicOnline #7
Conversation
There's a really hacky way:
I had unit tests in the old repo but haven't gotten around to adding the setup here yet. Regardless this seems to work as the code describes, but I had one question: you're always prefixing the "TPB" chapters with 0, but I think they can have multiple?
I don't know much about this site or western comic numbering, so correct me if I'm wrong! |
Thanks, I'll give this a test when i get the chance. I originally had it working the way you described with it accounting for each "Part" of the TPB. But considering that each part is basically the same chapter (I'm not sure why ReadComicOnline splits them up, they probably have their reasons) I settled on treating them as the same chapter. Let me know if you disagree with this, I'm happy to revert back to accounting for each part. As for prefixing with a "0", in the ideal world it would just be 1.1, 1.2, 2.1, 2.2 (or without the decimals according to my solution) but annoyingly some comics have a mix of actual issues and TPB's: https://readcomiconline.li/Comic/Invincible - so I'm having to ensure they appear before the actual chapter numbers. It isn't perfect but webscraping rarely is 😅 I don't think there is a set standard for western comic numbering and is often all over the place. I've been having issues on a personal project with issue numbers and it's always a pain to deal with lol. |
The main practical issue I think is that we can't refer to 2 subsequent chapters with the same chapterNumber. Houdoku wouldn't know whether it's the same chapter from a different scan, or a second part. So when users click "next chapter" it would get skipped. I think my preference would be:
This would cause an overlap if e.g. Issue 7 and TPB 7 both exist, but the groupName should allow users to filter those versions, and when they click "next chapter" it should stay with the same group. |
Right yes this should work - I wasn't actually aware of the |
Without these chapter numbers, certain functionality breaks in the Houdoku app (e.g. unread chapter count). It's not the best solution but I wasn't sure what else to do. Let me know if there could be a better solution.
I've not been able to test it properly as I weren't sure how to get it loaded into Houdoku (couldn't find any instructions unless I've missed something).