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

FlameComics Fix#23

Closed
ReCoffee wants to merge 2 commits into
AppFuton:masterfrom
ReCoffee:patch-1
Closed

FlameComics Fix#23
ReCoffee wants to merge 2 commits into
AppFuton:masterfrom
ReCoffee:patch-1

Conversation

@ReCoffee

Copy link
Copy Markdown

Refactor parseManga to handle missing series_id and improve filtering logic.

Refactor parseManga to handle missing series_id and improve filtering logic.
@ReCoffee

Copy link
Copy Markdown
Author

i hope this will be work

@ReCoffee ReCoffee changed the title Refactor manga parsing and filtering logic FlameComics Fix Mar 1, 2026
@LandWarderer2772 LandWarderer2772 requested a review from Copilot March 4, 2026 05:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Refactors the FlameComics source parser to be more resilient to incomplete series JSON payloads (notably missing series_id) and to tighten up list filtering and chapter URL handling.

Changes:

  • Update parseManga to return null when series_id is missing, skipping invalid items in list results and throwing a clearer error in details.
  • Adjust list filtering pipeline to handle nullable parseManga results and keep sorting logic intact.
  • Harden chapter URL parsing and make tag extraction more defensive for malformed categories payloads.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…lameComics.kt

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@LandWarderer2772

Copy link
Copy Markdown
Member

There's some error so I'll debug and then merge later

@ReCoffee

ReCoffee commented Mar 4, 2026

Copy link
Copy Markdown
Author

There's some error so I'll debug and then merge later

Ok

@ReCoffee ReCoffee closed this Mar 19, 2026
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.

3 participants