Skip to content

refactor: pluralize StableFX Settlement Advance API paths#454

Merged
anilgangapersaud-circle merged 2 commits into
circlefin:masterfrom
yitianx00:claude/quizzical-wiles-83e6ee
Jun 30, 2026
Merged

refactor: pluralize StableFX Settlement Advance API paths#454
anilgangapersaud-circle merged 2 commits into
circlefin:masterfrom
yitianx00:claude/quizzical-wiles-83e6ee

Conversation

@yitianx00

Copy link
Copy Markdown
Contributor

Summary

Mirrors wallets-api#3485, which pluralizes the StableFX Settlement Advance external collection resources per REST convention. Updates the sample app's outbound request paths in lib/stablefxTradesApi.ts in lockstep.

Before After Affects
/settlementAdvance /settlementAdvances list + create + GET /{advanceId}
…/reservation/{id}/cancel …/reservations/{id}/cancel cancel reservation
/signatures/settlementAdvance/presign /signatures/settlementAdvances/presign signature presign
…/repayment …/repayments create repayment + by-id
…/credit unchanged per-maker singleton

Verb sub-paths (reserve, cancel) are unchanged. The /debug/stablefx/settlementAdvance/... Nuxt page routes are local frontend paths, not backend API calls, so they are left as-is.

Test plan

  • BASE_URL=https://api-smokebox.circle.com — exercise SA flows against smokebox

🤖 Generated with Claude Code

yitianx00 and others added 2 commits June 30, 2026 10:21
Mirrors wallets-api#3485, which pluralizes the StableFX Settlement
Advance external collection resources per REST convention. Updates the
sample app's outbound request paths in lockstep.

- /settlementAdvance -> /settlementAdvances (list/create/by-id)
- .../reservation/{id}/cancel -> .../reservations/{id}/cancel
- /signatures/settlementAdvance/presign -> /signatures/settlementAdvances/presign
- .../repayment -> .../repayments (create + by-id)

/credit and the reserve/cancel verb sub-paths are unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@anilgangapersaud-circle
anilgangapersaud-circle merged commit cf307b5 into circlefin:master Jun 30, 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