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

[FEATURE REQUEST] Filtering Out System Notifications with callbackBeforeSendSingleMsgCommand webhook #2282

Closed
armolee opened this issue May 8, 2024 · 2 comments · Fixed by #2703
Assignees

Comments

@armolee
Copy link

armolee commented May 8, 2024

Why this feature?

I am writing to request a new feature regarding the callbackBeforeSendSingleMsgCommand webhook. Currently, after setting up this webhook, my API endpoint constantly receives system notifications, as exemplified below:
{ "SendID":7826049878, "CallbackCommand":"callbackBeforeSendSingleMsgCommand", "ServerMsgID":"c7358d02d5c068785192e9d96d80e814", "ClientMsgID":"ca9f6c9862cf381c83e80a0411eaf08b", "SenderPlatformID":0, "SenderNickname":"", "SessionType":1, "MsgFrom":200, "ContentType":1304, "Status":0, "CreateTime":1715135597256, "Content":"", "Seq":0, "AtUserList":[], "FaceURL":"", "Ex":"", "RecvID":8465386380 }

Suggested Solution

As per my requirements, I am only interested in capturing the events when a user manually sends a message.

I would greatly appreciate if a feature could be implemented that enables filtering out these system notifications. This would significantly help in making the webhook more efficient and aligned with my needs.

Thank you for considering my request. I look forward to hearing from you.

Additional Information

No response

@kubbot
Copy link
Contributor

kubbot commented May 8, 2024

Hello! Thank you for filing an issue.

If this is a bug report, please include relevant logs to help us debug the problem.

Join slack 🤖 to connect and communicate with our developers.

@printlin
Copy link
Contributor

Hello, I previously submitted a PR2370 that implements message callbacks only for certain IDs. You can refer to the corresponding changes.

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 a pull request may close this issue.

4 participants