slacker icon indicating copy to clipboard operation
slacker copied to clipboard

feat: exposing request from event in interaction handler in order to Ack event with custom payload

Open Vladmel1234 opened this issue 1 year ago • 6 comments

this PR addressing https://github.com/slack-io/slacker/issues/22

Vladmel1234 avatar Sep 16 '24 14:09 Vladmel1234

There is no much sense in first commit without the second one

Vladmel1234 avatar Sep 18 '24 15:09 Vladmel1234

Would love to get @arusso 's opinion on this PR.

raed-shomali avatar Sep 18 '24 15:09 raed-shomali

I'll fix the workflow error, the line length shouldn't prevent progression.

I'll try and look at this later today. I have questions but need to dig in a bit to make sure they're not self explanatory.

arusso avatar Sep 18 '24 16:09 arusso

btw if it is needed to add an example i will happily add one

Vladmel1234 avatar Sep 24 '24 18:09 Vladmel1234

Examples are always welcome!

raed-shomali avatar Oct 02 '24 00:10 raed-shomali

@arusso Let us know when you have had the chance to review the PR.

raed-shomali avatar Oct 02 '24 00:10 raed-shomali