Skip to content
This repository was archived by the owner on May 14, 2026. It is now read-only.

Fixes/hstream: Add series grouping#1333

Merged
Arkai1 merged 13 commits into
Kohi-den:mainfrom
PineappleTwilight:fixes/hstream
May 9, 2026
Merged

Fixes/hstream: Add series grouping#1333
Arkai1 merged 13 commits into
Kohi-den:mainfrom
PineappleTwilight:fixes/hstream

Conversation

@PineappleTwilight

@PineappleTwilight PineappleTwilight commented May 3, 2026

Copy link
Copy Markdown
Contributor

Checklist:

  • Updated extVersionCode value in build.gradle for individual extensions
  • Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
  • Referenced all related issues in the PR body (e.g. "Closes #xyz")
  • Added the isNsfw = true flag in build.gradle when appropriate
  • Have not changed source names
  • Have explicitly kept the id if a source's name or language were changed
  • Have tested the modifications by compiling and running the extension through Android Studio
  • Have removed web_hi_res_512.png when adding a new extension
  • Have made sure all the icons are in png format

This PR aims to add episode grouping to hstream. This is a toggleable option so users can switch back to the old format if they wish.

Also fixed episode dates to use the release date instead of the site upload date.

This will be the primary repository for my PRs moving forward in lieu of Yuzono due to much faster PR review times and overall higher quality code.

Amnesia added 8 commits May 2, 2026 20:06
Replace keiyoushi.utils imports with Aniyomi-compatible equivalents:
- parseAs → eu.kanade.tachiyomi.util.parseAs
- getPreferencesLazy → Injekt lazy SharedPreferences pattern
- addSwitchPreference → manual SwitchPreferenceCompat pattern

Fix SharedPreferences extension method calls (getBoolean, getString, edit)
Add kotlin-test and okhttp test dependencies
Move test file from src/test/ to test/kotlin/ for correct source set
- Changed ungrouped path selector from fa-upload to fa-regular fa-calendar
- Added release date fetching in grouped path for all episode paths
- Replaced all date_upload = 0L assignments with actual release date lookups
- Episode dates now reflect original release/air date, not site upload date
… date text is inside a <p> child element within the <a> tag, so ownText() returns empty. text() correctly includes descendant text.
@Arkai1

Arkai1 commented May 3, 2026

Copy link
Copy Markdown
Collaborator

Why Tests, There isn't any need for that.

@PineappleTwilight

Copy link
Copy Markdown
Contributor Author

Why Tests, There isn't any need for that.

I can remove them if it's really a problem, otherwise theyre harmless I'd think

@Arkai1

Arkai1 commented May 4, 2026

Copy link
Copy Markdown
Collaborator

Why Tests, There isn't any need for that.

I can remove them if it's really a problem, otherwise theyre harmless I'd think

There isn't any need for them so might as well remove them from build.gradle as well

@PineappleTwilight

Copy link
Copy Markdown
Contributor Author

Why Tests, There isn't any need for that.

I can remove them if it's really a problem, otherwise theyre harmless I'd think

There isn't any need for them so might as well remove them from build.gradle as well

Done.

@PineappleTwilight

Copy link
Copy Markdown
Contributor Author

Bump in case maintainers didn't see my edit

@Arkai1

Arkai1 commented May 8, 2026

Copy link
Copy Markdown
Collaborator

Have You check everything, Search, filters, video list, episode list and That by enabling group it doesn't duplicate entries. Plus only check the task in hecklist that you did there isn't any need to check them all

@PineappleTwilight

Copy link
Copy Markdown
Contributor Author

Have You check everything, Search, filters, video list, episode list and That by enabling group it doesn't duplicate entries. Plus only check the task in hecklist that you did there isn't any need to check them all

I mean, yes due to the way it works there's no way it could duplicate entries. I'll double-check and reply again if everything is good.

Amnesia added 3 commits May 8, 2026 20:24
- Fixed null safety in HstreamFilters.kt (removed !! operators)
- Improved error messages (XSRF-TOKEN, date parsing)
- Clarified 'Group by series' preference summary
- Added '(fallback)' indicator for single-episode mode
- Added Toast for ActivityNotFoundException
- Added episode sort order preference
- Used robust extractEpisodeNumber() for URL parsing
- Deduplication fix for episode grouping
- Fixed context reference in HstreamUrlActivity (ActivityNotFoundException handler)
@PineappleTwilight

Copy link
Copy Markdown
Contributor Author

Have You check everything, Search, filters, video list, episode list and That by enabling group it doesn't duplicate entries. Plus only check the task in hecklist that you did there isn't any need to check them all

Everything is checked and confirmed. Ready to merge.

@Arkai1
Arkai1 merged commit a1b15e2 into Kohi-den:main May 9, 2026
5 checks passed
@PineappleTwilight
PineappleTwilight deleted the fixes/hstream branch May 10, 2026 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants