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

修复获取小说文字接口并新增几条 API #399

Merged
merged 6 commits into from
Mar 9, 2024

Conversation

asadahimeka
Copy link
Contributor

v1/novel/text 已被官方移除,改为调用 webview/v2/novel 作兼容处理

新增以下几条 API:

  • 获取小说 HTML 后解析 JSON
  • 相关小说作品
  • 相关用户
  • 漫画系列
  • 用户的漫画系列
  • 用户的小说系列
  • 人气直播列表

Copy link

codecov bot commented Mar 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.09%. Comparing base (468c98c) to head (eda29c0).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
+ Coverage   91.06%   91.09%   +0.02%     
==========================================
  Files          58       58              
  Lines        1870     1898      +28     
==========================================
+ Hits         1703     1729      +26     
- Misses        167      169       +2     
Files Coverage Δ
hibiapi/api/pixiv/api.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 468c98c...eda29c0. Read the comment docs.

Copy link
Member

@mnixry mnixry left a comment

Choose a reason for hiding this comment

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

感谢PR!有几个小问题修改一下就好了

没想到测试挂了这么多,我过会得看看叔叔的站API发生什么了……

@mnixry
Copy link
Member

mnixry commented Mar 9, 2024

测试挂了一个,你看着修一下,过了就行

Typing的问题我来修吧

@asadahimeka
Copy link
Contributor Author

测试挂了一个,你看着修一下,过了就行

Typing的问题我来修吧

novel_text 忘了删 raw 参数了😹 改了

Copy link
Member

@mnixry mnixry left a comment

Choose a reason for hiding this comment

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

LGTM

@mnixry mnixry merged commit 0c91dd9 into mixmoe:main Mar 9, 2024
4 of 23 checks passed
@mnixry mnixry added bug Something isn't working enhancement New feature or request labels Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants