Skip to content

docs: add rule-based accounts guide and document USDC and USDT support - #1243

Merged
bsiaotickchong merged 3 commits into
mainfrom
10-27-rule-based-accounts-guide
Oct 30, 2026
Merged

bsiaotickchong merged 3 commits into
mainfrom
10-27-rule-based-accounts-guide

Conversation

@bsiaotickchong

@bsiaotickchong bsiaotickchong commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds a concise Rule-Based Accounts guide at payouts-and-b2b/depositing-funds/rule-based-accounts, next to Internal Accounts in the "Managing accounts" nav group. It covers supported currencies, prerequisites, creating an account (USD and USDC tabs), sweep rule fields, tracking sweeps and SWEEP_* failures, and sandbox testing.
  • Documents USDC and USDT as supported currencies for POST /internal-accounts. The currency field and the endpoint description previously said rule-based accounts were USD only. Adds a usdcDepositAddress request example.
  • Links the new guide from the Internal Accounts page's next steps.

Notes

  • A USDC rule-based account returns one deposit address per network the platform supports in fundingPaymentInstructions, not a bank account number. USDC and USDT rule-based accounts are enabled per platform, separately from USD.

Test plan

  • make build and make lint pass.
  • Rendered the new page locally with mint dev (4.2.284):

Rule-based accounts page

Requested by @bsiaotickchong

Original PR: #1242

@vercel

vercel Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated
grid-cards-demo Ignored Ignored Preview Oct 29, 2026 10:54pm UTC
grid-flow-builder Ignored Ignored Preview Oct 29, 2026 10:54pm UTC
grid-wallet-demo Ignored Ignored Preview Oct 29, 2026 10:54pm UTC

Request Review

Copy link
Copy Markdown

This stack of pull requests is managed by Graphite. Learn more about stacking.

@mintlify

mintlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
Grid 🟢 Ready View Preview Oct 29, 2026, 10:55 PM

@greptile-apps

greptile-apps Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[Low impact] The PR changes account documentation without changing runtime behavior.

The PR appears safe to merge, with a non-blocking request to document USDC platform enablement in the guide.

Findings

  1. P2 USDC prerequisite is missing ▶
Fix with agent prompt
### Issue 1
mintlify/payouts-and-b2b/depositing-funds/rule-based-accounts.mdx:23
The guide omits that USDC rule-based accounts must be enabled separately for your platform. If a platform lacks that enablement, readers can meet every listed prerequisite and still have account creation rejected. The endpoint description already calls out this requirement; add it here before the creation examples.

```suggestion
- USDC rule-based accounts must be enabled separately for your platform.
- The account holder already has an internal account in the same currency. Funds go there when a sweep can't be completed.
```

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

Adds a rule-based accounts guide and documents USDC support in the OpenAPI source and both bundles.

  • Developers can follow a guide to set up rule-based accounts.
  • The account API docs now describe USDC deposit addresses.

Reviews (1) · Last reviewed commit: "docs: add rule-based accounts guide and ..." · Reviewed by Greptile


## Prerequisites

- The account holder already has an internal account in the same currency. Funds go there when a sweep can't be completed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 USDC prerequisite is missing

The guide omits that USDC rule-based accounts must be enabled separately for your platform. If a platform lacks that enablement, readers can meet every listed prerequisite and still have account creation rejected. The endpoint description already calls out this requirement; add it here before the creation examples.

Suggested change
- The account holder already has an internal account in the same currency. Funds go there when a sweep can't be completed.
- USDC rule-based accounts must be enabled separately for your platform.
- The account holder already has an internal account in the same currency. Funds go there when a sweep can't be completed.
Prompt To Fix With AI
This is a comment left during a code review.
Path: mintlify/payouts-and-b2b/depositing-funds/rule-based-accounts.mdx
Line: 23

Comment:
**USDC prerequisite is missing**

The guide omits that USDC rule-based accounts must be enabled separately for your platform. If a platform lacks that enablement, readers can meet every listed prerequisite and still have account creation rejected. The endpoint description already calls out this requirement; add it here before the creation examples.

```suggestion
- USDC rule-based accounts must be enabled separately for your platform.
- The account holder already has an internal account in the same currency. Funds go there when a sweep can't be completed.
```

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaving this out on purpose: the PR owner asked for the platform-enablement note to be removed from the guide, and the endpoint reference still covers it.


🤖 reinforced-asteroid-v6(#1) | Feedback

@ls-bolt

ls-bolt Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

⚡ Review ledger

Round 1

  1. USDC prerequisite (platform enablement) missing from guide → NOT APPLICABLE: removed at the PR owner's request; the endpoint description still states it

Round 2

  1. Is USDT / USDB supported? → ANSWERED: USDT is gated and USDB-sweep-only, so it's left out pending reviewer's call; USDB isn't a rule-based account currency

Round 3

  1. USDT / USDB follow-up (@bsiaotickchong) → NOT APPLICABLE: guide stays USD + USDC; USDT documentation deferred to the USDT → USDB work

Round 4

  1. Why would USDT need new flow runs? (@jklein24) → ANSWERED: paycore already accepts USDT; the USDB-only limit is a sparkcore validation plus gate. No docs change

| Currency | What payers send to |
|----------|---------------------|
| `USD` | A dedicated US bank account and routing number |
| `USDC` | A dedicated deposit address on each supported network for your platform (for example, Solana or Base) |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is USDT not supported? Why not if USDC is? What about USDB?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

USDT works, but only behind a separate gate and only when it sweeps into the holder's own USDB embedded wallet. I left it out because that's a much narrower case than USDC. USDB isn't accepted as a rule-based account currency (it returns a 400); it only appears as a sweep destination. Want me to add a USDT row that states the USDB-only restriction?


🤖 reinforced-asteroid-v6(#1) | Feedback

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah USDT to USDB was what @kphurley7 was working on i think?

basically I was just prioritizing USDC with my work, additional currencies just need additional flow runs i think but can be added

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, so I'm keeping the guide to USD and USDC for now. A USDT row can be added alongside the USDT → USDB work once it's ready to document.


🤖 reinforced-asteroid-v6(#1) | Feedback

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would it need new flow runs if it's just the asset changing? I think all of the existing fireblocks actions can take any asset param

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the code, paycore's rule-based deposit-address creation already accepts both USDC and USDT. The limit is a check in sparkcore's rule_based_accounts.py that only lets USDT sweep into the holder's USDB wallet, behind GRID_RULE_BASED_USDB_SWEEPS_ENABLED. Widening USDT looks like lifting that check, not adding new flow runs, though Brian may know of a settlement gap I can't see.


🤖 reinforced-asteroid-v6(#1) | Feedback

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah i spoke too soon i'll look into removing that restriction

@ls-bolt

ls-bolt Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

🦣 Congratulations @jklein24 - your substantive review earned a Siberian unicorn! (rare)

This giant grazing rhinoceros carried a huge domed boss that supported a horn.

View your Frost-dex: https://zeus.dev.dev.sparkinfra.net/#/dinodex/jklein24?section=ice-age

@ls-bolt

ls-bolt Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

📌 Bolt Status

2026-10-28 00:46:13 UTC — ⚡ Agent reinforced-asteroid-v6-2(#2) started


Feedback

bsiaotickchong and others added 2 commits October 29, 2026 22:21
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: bsiaotickchong <bsiaotickchong@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ls-bolt
ls-bolt Bot force-pushed the 10-27-rule-based-accounts-guide branch from 548fd24 to 6d1ebbf Compare October 29, 2026 22:22
@ls-bolt ls-bolt Bot changed the title docs: add rule-based accounts guide and document USDC support docs: add rule-based accounts guide and document USDC and USDT support Oct 29, 2026
@ls-bolt

ls-bolt Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

⚡ Review rounds — updated in place, latest first.

Round 2 · 1fa8788

  • Removed the USDT sweep-destination limit from the guide, the currency field, the endpoint description, and the 400 reasons, per @bsiaotickchong
  • Refreshed the PR description screenshot
  • Verified: make build and make lint pass; removal-only change, so no adversarial review rerun
Earlier rounds (1)

Round 1 · 6d1ebbf

Commits

  • c5a1168 docs: add rule-based accounts guide and document USDC support

  • 6d1ebbf docs: document USDT rule-based accounts

  • Rebased onto main

  • Documented USDT rule-based accounts in the guide's currency table and in POST /internal-accounts (currency field, endpoint description, 400 reasons), including which destinations a USDT account can sweep to

  • Verified: make build and make lint pass; adversarial review found nothing actionable

Co-Authored-By: bsiaotickchong <bsiaotickchong@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bsiaotickchong
bsiaotickchong merged commit 83ca320 into main Oct 30, 2026
9 checks passed
@bsiaotickchong
bsiaotickchong deleted the 10-27-rule-based-accounts-guide branch October 30, 2026 00:34

This branch was successfully deployed

1 active deployment
staging - mintlify — 1fa87884 Deployed Oct 29, 2026 by mintlify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants