nonebot-bison icon indicating copy to clipboard operation
nonebot-bison copied to clipboard

:sparkles: 适配小刻食堂平台

Open AzideCupric opened this issue 1 year ago • 3 comments

AzideCupric avatar Sep 10 '23 13:09 AzideCupric

Deploy Preview for nonebot-bison ready!

Name Link
Latest commit abbea5fcf7ddba6551ce0f6b5906172b306de57c
Latest deploy log https://app.netlify.com/sites/nonebot-bison/deploys/669157acf057110008ed9c5e
Deploy Preview https://deploy-preview-379--nonebot-bison.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] avatar Sep 10 '23 13:09 netlify[bot]

Codecov Report

Attention: Patch coverage is 77.73512% with 116 lines in your changes missing coverage. Please review.

Project coverage is 85.04%. Comparing base (e089e87) to head (fd66299). Report is 2 commits behind head on main.

Files Patch % Lines
nonebot_bison/platform/ceobecanteen/platform.py 71.78% 57 Missing :warning:
nonebot_bison/theme/themes/ceobe_canteen/build.py 54.43% 36 Missing :warning:
nonebot_bison/utils/image.py 42.85% 8 Missing :warning:
nonebot_bison/theme/utils.py 68.42% 6 Missing :warning:
nonebot_bison/platform/ceobecanteen/cache.py 95.38% 3 Missing :warning:
nonebot_bison/platform/ceobecanteen/utils.py 80.00% 3 Missing :warning:
nonebot_bison/theme/themes/arknights/build.py 76.92% 3 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #379      +/-   ##
==========================================
+ Coverage   84.75%   85.04%   +0.28%     
==========================================
  Files          81       88       +7     
  Lines        3996     4486     +490     
==========================================
+ Hits         3387     3815     +428     
- Misses        609      671      +62     
Flag Coverage Δ
smoke-test 85.04% <77.73%> (+0.28%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 10 '23 13:09 codecov[bot]

整理一下还需要讨论的问题:在产品层面需要解决两个问题,1. 可以在订阅的时候看到有哪些数据源是可以订阅的;2. 最好提供一些与定义的订阅组,可以让用户一键式订阅预定义订阅源。

这样可以考虑的路线分为两条:

第一个是 ceobecanteen 作为有 target 的 platform,通过订阅不同的 target 来完成订阅不同的数据源。如果使用这条路线,那么需要解决下面几个问题:

  1. 用户需要自己输入数据源
  2. 每次只能订阅一个,如果在此引入订阅组代价可能比较高

第二个是 ceobecanteen 作为无 target 的 platform,通过选择 categories 来完成。如果使用这条路线,那么需要解决下面几个问题:

  1. 现在可选择的数据源很多,如果全部展示出来好像确实太长了
  2. 现在 Category 的类型是 int,需要进行拓展
  3. 不能用一个 Category 覆盖多个 Category,可能需要实现一种 post 同时属于多个 Category

felinae98 avatar Mar 07 '24 07:03 felinae98