Skip to content

fix(douban): fix marks pagination and improve subject data extraction#752

Merged
jackwener merged 1 commit intojackwener:mainfrom
ykfnxx:fix/douban-marks-pagination
Apr 4, 2026
Merged

fix(douban): fix marks pagination and improve subject data extraction#752
jackwener merged 1 commit intojackwener:mainfrom
ykfnxx:fix/douban-marks-pagination

Conversation

@ykfnxx
Copy link
Copy Markdown
Contributor

@ykfnxx ykfnxx commented Apr 4, 2026

Summary

  • fix douban marks pagination by matching the actual 15-item grid page size
  • split combined subject titles into title and originalTitle when possible
  • return normalized country and numeric duration fields from douban subject pages

Testing

  • npm test
  • npm run build

1. marks: correct pageSize from 30 to 15 — douban grid mode shows 15
   items per page, causing pagination to stop after the first page.

2. subject: split title/originalTitle correctly — v:itemreviewed contains
   both Chinese and original titles concatenated.

3. subject: extract country/region from #info as list, split by "/".

4. subject: extract duration as pure number (min) from v:runtime or #info.

5. subject: return casts as list instead of comma-joined string.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jackwener jackwener merged commit 1bcd96f into jackwener:main Apr 4, 2026
11 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