-
Notifications
You must be signed in to change notification settings - Fork 680
Open
Labels
auto-triage-skipenhancementM-T: A feature request for new functionalityM-T: A feature request for new functionalitypkg:oauthapplies to `@slack/oauth`applies to `@slack/oauth`
Description
Currently, the installer.handleCallback method supports the Node specific IncomingMessage and ServerResponse. This is not ideal for apps using Bun and other frameworks like Hono:
Here,
c.req.rawis of typeRequestand is not assignable toIncomingResponsewhich is anode:httpspecific type.c.Res(type ofResponse) is not assignable toServerResponse<IncomingMessage>.
I understand that this is node-slack-sdk and not bun-slack-sdk but this feature request is more about adopting a more compatible standard hence making it more framework/runtime agnostic. Some kind of backward compatibility might be needed.
If this is out of scope, how can I use this package with my above example (Hono and Bun)?
Packages:
Select all that apply:
-
@slack/web-api -
@slack/rtm-api -
@slack/webhooks -
@slack/oauth -
@slack/socket-mode -
@slack/types - I don't know
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
Metadata
Metadata
Assignees
Labels
auto-triage-skipenhancementM-T: A feature request for new functionalityM-T: A feature request for new functionalitypkg:oauthapplies to `@slack/oauth`applies to `@slack/oauth`