Skip to content

feat: add delete listen key support#94

Closed
Akur1a wants to merge 1 commit into
singlesly:mainfrom
Akur1a:fix/delete-listen-key
Closed

feat: add delete listen key support#94
Akur1a wants to merge 1 commit into
singlesly:mainfrom
Akur1a:fix/delete-listen-key

Conversation

@Akur1a
Copy link
Copy Markdown

@Akur1a Akur1a commented May 11, 2026

Summary

  • add a BingxDeleteListenKeyEndpoint for deleting user data stream listen keys
  • expose deleteListenKey(account, listenKey) on ListenKeyService
  • add a focused unit test for the delete listen key service path

Verification

  • npx jest src/bingx-client/services/listen-key.service.spec.ts --runInBand
  • npx eslint src/bingx/endpoints/bingx-delete-listen-key-endpoint.ts src/bingx/endpoints/index.ts src/bingx-client/services/listen-key.service.ts src/bingx-client/services/listen-key.service.spec.ts
  • npm run build
  • npx jest src/bingx-client/bingx-api.client.spec.ts src/bingx-client/services/listen-key.service.spec.ts src/nest-bingx/nest-bingx.module.spec.ts --runInBand

Note

npm test -- --runInBand timed out locally after the unit suites started; the playground specs require live API credentials and the socket specs are long-running in this environment. The focused service tests and build pass.

Closes #25

@Akur1a Akur1a force-pushed the fix/delete-listen-key branch from 071f216 to 45a8767 Compare May 11, 2026 08:14
@Akur1a
Copy link
Copy Markdown
Author

Akur1a commented May 11, 2026

Closing this because I noticed there is already an earlier PR covering the same issue (#77). Thanks.

@Akur1a Akur1a closed this May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement deletion listen key

1 participant