Skip to content

fix: correct account id field and add EUR for DAA deposit addresses - #481

Merged
anilgangapersaud-circle merged 2 commits into
circlefin:masterfrom
yitianx00:claude/digital-asset-endpoint-review-f283c8
Sep 14, 2026
Merged

anilgangapersaud-circle merged 2 commits into
circlefin:masterfrom
yitianx00:claude/digital-asset-endpoint-review-f283c8

Conversation

@yitianx00

Copy link
Copy Markdown
Contributor

Summary

  • The create-account-deposit-address and list-deposit-addresses debug flows sent clientEntityId in the request, but per the Circle API docs the field is accountId (clientEntityId is only valid on the Create Account endpoint).
  • The deposit address create form only offered USD; the API also supports EUR.

Test plan

  • vue-tsc --noEmit passes
  • Ran the app locally against smokebox and confirmed the create page shows "Account Id (optional)" and a USD/EUR currency dropdown

🤖 Generated with Claude Code

The deposit address create/list endpoints were sending clientEntityId
instead of accountId, and the create form only offered USD, both out
of sync with the Circle API spec.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@anilgangapersaud-circle
anilgangapersaud-circle merged commit 7338f14 into circlefin:master Sep 14, 2026
4 checks passed
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.

2 participants