-
Notifications
You must be signed in to change notification settings - Fork 4
feat: Discord Integration with Send Text Message #1728
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
base: main
Are you sure you want to change the base?
feat: Discord Integration with Send Text Message #1728
Conversation
|
/superplane fix-formatting |
shiroyasha
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/superplane fix-formatting
|
aaaa |
|
/superplane fix-formatting |
aa9fe0a to
ce9bfe1
Compare
We can, and most likely we will. The approach that i've started is compatible with the HTTP API. The outgoing webhook (as implemented) allows us to send messages, while the incoming webhooks (part of the Interactions Endpoint, which is part of the HTTP API) allow us to listen to slash commands, on button clicks, etc. The only thing (that I've found) it doesnt listen to is pure text messages (that do not contain a slash command). I thought this wouldnt be necessary for our case. If you want i can implement the incoming webhooks within this PR, but they wont be testable without a particular trigger. I propose we make another task (like "[Discord] onSlashCommand") in which we will do both the trigger and expand the Discord App Integration |
2c9f9ff to
86c6fb8
Compare
Signed-off-by: Emil Todorovski <emil.todorovski@brightmarbles.io>
Signed-off-by: Emil Todorovski <emil.todorovski@brightmarbles.io>
Signed-off-by: Emil Todorovski <emil.todorovski@brightmarbles.io>
7aea129 to
e675946
Compare
Signed-off-by: Emil Todorovski <emil.todorovski@brightmarbles.io>
Decisions:
UI: