Skip to content
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

character item switch to next api #409

Closed
wants to merge 2 commits into from
Closed

character item switch to next api #409

wants to merge 2 commits into from

Conversation

ErBWs
Copy link
Contributor

@ErBWs ErBWs commented Nov 19, 2024

现在有一个冲突,next api 没有声优的 short summary。

我在想是否可以利用 /v0/characters/{character_id} 获取角色介绍来代替,同时这里也能获得角色的中文名

image

@Predidit
Copy link
Owner

bangumi/server#662 已经合并,虽然距离部署还有一段时间,但这部分现在似乎没有必要再迁移到不保持稳定的 next api

@Predidit
Copy link
Owner

我们似乎只需要从 large 迁移到 grid 并等待 bangumi 后端的更新就可以了

@ErBWs
Copy link
Contributor Author

ErBWs commented Nov 19, 2024

grid 现在已经正常返回头图了

不过切换到 next api 还有一个原因就是 v0 api 获取到的列表顺序会出现先配角再主角的情况,比如影之实力者1、2季、村井之恋等。我不清楚为什么会和网页上有差异

@Predidit
Copy link
Owner

next API 现在缺乏 short summary 。

额外的请求不是好主意

@ErBWs
Copy link
Contributor Author

ErBWs commented Nov 19, 2024

啊。了解了

@ErBWs ErBWs closed this Nov 19, 2024
@Predidit
Copy link
Owner

也许我们可以在另一个PR里从 large 迁移到 grid

@ErBWs
Copy link
Contributor Author

ErBWs commented Nov 19, 2024

也许我们可以在另一个PR里从 large 迁移到 grid

这个修改我认为不需要 PR,仓库所有者直接对仓库本体进行 commit 会更加便捷

@Predidit
Copy link
Owner

已经完成

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