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

Update guessmusic plugin #898

Merged
merged 1 commit into from
May 7, 2024
Merged

Update guessmusic plugin #898

merged 1 commit into from
May 7, 2024

Conversation

vatebur
Copy link
Member

@vatebur vatebur commented May 7, 2024

  1. 修复:正确答案变小写、简体的问题
    原先是把标准答案转小写、简体,然后再和输入答案比较。现在是每次比较的时候,都对标准答案和输入答案进行转换。是否要再加个变量先把转换后的标准答案存起来,比较的时候用。

  2. 抽选歌曲的时候,只抽选本地已经下载好的歌曲
    原先抽歌的时候有概率会调用API,改成只从下载好的歌曲里选取

  3. 将插件改为默认禁用
    因为下载的API失效了,不过依然可以通过其他方式添加歌曲

  4. 新功能:提供猜歌选项
    如图
    image

1.修复:正确答案变小写的问题。
2.抽选歌曲的时候,只抽选本地已经下载好的歌曲
3.将插件改为默认禁用
4.新功能:提供猜歌选项
@vatebur
Copy link
Member Author

vatebur commented May 7, 2024

lint的检查没过,但是我不知道问题出在哪

Copy link
Member

@fumiama fumiama left a comment

Choose a reason for hiding this comment

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

并非你的问题。

@fumiama fumiama merged commit d832f9a into FloatTech:master May 7, 2024
7 of 8 checks passed
YumeMichi added a commit to YumeMichi/ZeroBot-Plugin that referenced this pull request May 15, 2024
* https://github.com/FloatTech/ZeroBot-Plugin:
  feat: add plugin 抽扑克牌 (FloatTech#906)
  fix(workflow): pr not work properly
  Revert "feat: add plugin 抽扑克! (FloatTech#903)" (FloatTech#905)
  feat: add plugin 抽扑克! (FloatTech#903)
  [dish] fix: 关键词搜寻功能 (FloatTech#904)
  修复 FullMatchRule 中调用 ctx.NickName() 的 panic 问题 (FloatTech#902)
  optimize: gif plugin for image cache (FloatTech#901) (fix FloatTech#900)
  Update guessmusic plugin (FloatTech#898)
  make lint happy
  add robbery plugin (FloatTech#893)
  [mcfish]  Add one click selling (FloatTech#894)
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