-
Notifications
You must be signed in to change notification settings - Fork 11.5k
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
feat(Discord Node): New sendAndWait operation #12894
feat(Discord Node): New sendAndWait operation #12894
Conversation
…discord-sendandwait-operation
Codecov ReportAttention: Patch coverage is 📢 Thoughts on this report? Let us know! |
…discord-sendandwait-operation
…discord-sendandwait-operation
…discord-sendandwait-operation
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.
Follow up question, and I'm not sure if this is out of scope: Shouldn't we send some sort of confirmation after the user hits approval or decline?
@@ -20,6 +20,8 @@ export const description: INodeProperties[] = getSendAndWaitProperties( | |||
undefined, | |||
{ | |||
noButtonStyle: true, | |||
defaultApproveLabel: '✓ Approve', | |||
defaultDisapproveLabel: '✗ Decline', |
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.
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.
@dana-gill
by default message send with one button option, you need to chose corresponding Approval option
…discord-sendandwait-operation
…discord-sendandwait-operation
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.
typo :) And it looks like Codecov is complaining about tests for utils.ts
. If there could be more testing for that, that would be great 🙏
Co-authored-by: Dana <[email protected]>
…discord-sendandwait-operation
…hub.com/n8n-io/n8n into node-2325-discord-sendandwait-operation
✅ All Cypress E2E specs passed |
n8n Run #9061
Run Properties:
|
Project |
n8n
|
Branch Review |
node-2325-discord-sendandwait-operation
|
Run status |
Passed #9061
|
Run duration | 04m 29s |
Commit |
3d8e8e88c9: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 michael-radency 🗃️ e2e/*
|
Committer | Michael Kret |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
1
|
Pending |
5
|
Skipped |
0
|
Passing |
433
|
View all changes introduced in this branch ↗︎ |
Summary
New sendAndWait operation
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/NODE-2325/discord-sendandwait-operation