Skip to content

fix: 修复集合列表海报不显示#3123

Open
proffitteoy wants to merge 2 commits into
open-ani:mainfrom
proffitteoy:fix-subject-collection-cover-3107
Open

fix: 修复集合列表海报不显示#3123
proffitteoy wants to merge 2 commits into
open-ani:mainfrom
proffitteoy:fix-subject-collection-cover-3107

Conversation

@proffitteoy

Copy link
Copy Markdown
Contributor

Fixes #3107,我真等不及了每次看番个人列表都没封面显示,就动手修了一下。

在追 / 看过 / 想看等 subject collection 列表中,条目封面此前使用 Bangumi API 的图片重定向链接:

api.bgm.tv/v0/subjects/{id}/image

该链接在 Android 设备上可能出现超时,导致集合列表海报不显示;但 Ani 静态资源本身可以正常加载。

本 PR 将集合条目的封面地址改为 Ani 静态 subject cover URL,并在缓存的 collection entity 映射为 SubjectInfo 时进行规范化,避免已有本地缓存继续请求旧的 Bangumi 图片接口。

测试

  • ./gradlew.bat :app:shared:compileAndroidMain
  • Android debug install,确认新的 collection cover 请求使用 static.myani.org/bangumi/subjects/{id}/large

summary = summary,
nsfw = nsfw,
imageLarge = imageLarge,
imageLarge = staticSubjectImageLargeUrl(subjectId),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

这里感觉应该继续用 "imageLarge"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

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