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

Add more event data to event listener arguments; fix #1205 too #1317

Merged
merged 1 commit into from
Aug 27, 2021

Conversation

seratch
Copy link
Member

@seratch seratch commented Aug 26, 2021

Summary

This pull request adds more properties to the event listener arguments. The main motivation is to add retry num & retry reason to bolt-js's ReceiverEvent data as part of slackapi/bolt-js#759

This pull request fixes #1205 as it adds the first-ever unit tests to socket-mode package project.

Requirements (place an x in each [ ])

@seratch seratch added enhancement M-T: A feature request for new functionality pkg:socket-mode applies to `@slack/socket-mode` labels Aug 26, 2021
@seratch seratch added this to the socket-mode@1.2 milestone Aug 26, 2021
@seratch seratch self-assigned this Aug 26, 2021
@seratch
Copy link
Member Author

seratch commented Aug 26, 2021

Before merging this, we need to fix the errors with the latest TS compiler.

Error: src/KeepAlive.ts(221,45): error TS2571: Object is of type 'unknown'.
Error: src/RTMClient.ts(602,56): error TS2571: Object is of type 'unknown'.

@seratch seratch force-pushed the socket-mode-more-event-data branch from fba554c to 1580daf Compare August 27, 2021 03:49
@seratch
Copy link
Member Author

seratch commented Aug 27, 2021

I've rebased on the main branch. Once the CI builds pass, we will merge this PR.

@seratch seratch merged commit 90f3800 into slackapi:main Aug 27, 2021
@seratch seratch deleted the socket-mode-more-event-data branch August 27, 2021 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement M-T: A feature request for new functionality pkg:socket-mode applies to `@slack/socket-mode`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add unit tests for @slack/socket-mode
2 participants