Skip to content

feat(sms): support optional account for volcengine multi-account#128

Merged
tylor-zhang merged 3 commits into
mainfrom
feat/sms-multi-account
Jun 15, 2026
Merged

feat(sms): support optional account for volcengine multi-account#128
tylor-zhang merged 3 commits into
mainfrom
feat/sms-multi-account

Conversation

@tylor-zhang

Copy link
Copy Markdown
Contributor

Summary

  • Add optional account field to SendSmsDto for Volcengine SmsAccount (消息组 ID)
  • SmsService uses dto.account ?? VOLCENGINE_SMS_ACCOUNT when sending via volcengine
  • Persist account on SmsRecord for audit

Test plan

  • Unit tests: sms.service.spec.ts (dto account vs env fallback)
  • Deploy preview env and send domestic/foreign SMS with different accounts

Made with Cursor

tylor-zhang and others added 2 commits June 15, 2026 11:05
Allow callers to pass SmsAccount per request; fall back to VOLCENGINE_SMS_ACCOUNT when omitted.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

sdk version 2.27.1-pr-128-ef6200b85a

## for js/ts project
pnpm add @36node/auth-sdk@2.27.1-pr-128-ef6200b85a

Use resolveAccount when creating records so env fallback is stored.

Co-authored-by: Cursor <cursoragent@cursor.com>
@tylor-zhang tylor-zhang merged commit 18217cb into main Jun 15, 2026
3 checks passed
@tylor-zhang tylor-zhang deleted the feat/sms-multi-account branch June 15, 2026 07:40
tylor-zhang pushed a commit that referenced this pull request Jun 15, 2026
🤖 I have created a release *beep* *boop*
---


## [2.28.0](v2.27.1...v2.28.0)
(2026-06-15)


### Features

* **sms:** support optional account for volcengine multi-account
([#128](#128))
([18217cb](18217cb))


### Bug Fixes

* allow legacy weak old password when updating password
([#126](#126))
([f4d819f](f4d819f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

1 participant