Skip to content

Update all product names to just Grid - #10

Merged
jklein24 merged 1 commit into
mainfrom
10-10-update_all_product_names_to_just_grid
Oct 11, 2025
Merged

Update all product names to just Grid#10
jklein24 merged 1 commit into
mainfrom
10-10-update_all_product_names_to_just_grid

Conversation

@jklein24

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

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

@jklein24
jklein24 merged commit 65b82ed into main Oct 11, 2025
6 checks passed

Copy link
Copy Markdown
Contributor Author

Merge activity

@jklein24
jklein24 deleted the 10-10-update_all_product_names_to_just_grid branch October 11, 2025 07:02
jklein24 added a commit that referenced this pull request Jul 27, 2026
## Summary

Follow-up to the skill spec-refresh (#708). That PR removed the
response-only `paymentRails` field from the create examples in
`account-types.md`, but three references survived elsewhere and
contradicted the fix:

- `SKILL.md` — `"paymentRails": ["SPEI"]` in the external-account create
example
- `SKILL.md` — Best Practices tip #10 still asserted *"all fiat accounts
require `paymentRails`"*
- `references/workflows.md` — `"paymentRails": ["SPEI"]` and
`"paymentRails": ["ACH"]` in two create bodies

`paymentRails` is not an input on any `*ExternalAccountCreateInfo`
schema — Grid selects the rail and returns it on the created account
(`common/*AccountInfo.yaml`). Left as-is, the skill instructs callers to
send an invalid field on every fiat account create, and tip #10 is
high-weight guidance.

Tip #10 now points at the field that *is* required (`beneficiary`) and
states the rail is server-selected.

## Test plan

Docs-only. Verified no `paymentRails` remains in any request body across
the skill (the 3 surviving mentions all describe it as response-only),
and all 25 curl JSON bodies still parse.

Requested by @jklein24

Original PR: #752
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