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

[OneBot] 添加戳一戳二合一接口 #725

Merged
merged 2 commits into from
Jan 6, 2025

Conversation

Shua-github
Copy link
Contributor

send_poke
传入group_id按group_poke处理,
否则按friend_poke处理

@Shua-github Shua-github requested a review from ishkong January 5, 2025 12:06
Copy link
Collaborator

@Executor-Cheng Executor-Cheng left a comment

Choose a reason for hiding this comment

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

还是合并一下好了

Copy link
Collaborator

@Executor-Cheng Executor-Cheng left a comment

Choose a reason for hiding this comment

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

感谢贡献

@Executor-Cheng Executor-Cheng dismissed ishkong’s stale review January 6, 2025 06:15

Resolved in #6f1ae8d

@pk5ls20
Copy link
Contributor

pk5ls20 commented Jan 6, 2025

你的提交目前没有签名,无法被合并到主分支中,请补下签名
参考:https://docs.github.com/zh/authentication/managing-commit-signature-verification/signing-commits

@DarkRRb
Copy link
Contributor

DarkRRb commented Jan 6, 2025

你需要合并所有提交并为合并后的提交签名

因为要求是所有提交都要签名

传入group_id按group_poke处理,
否则按friend_poke处理

Update Lagrange.OneBot/Core/Operation/Message/SendPokeOperation.cs

Co-authored-by: DarkRRb <[email protected]>

忘删了

合并

失败返回-1并对提交签名
@Shua-github
Copy link
Contributor Author

你需要合并所有提交并为合并后的提交签名

因为要求是所有提交都要签名

应该可以了?

@pk5ls20 pk5ls20 merged commit 116c61f into LagrangeDev:master Jan 6, 2025
11 checks passed
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.

5 participants