docs: sync Grid Visualizer and Kotlin sample with Plasma/USDT schema changes - #771
docs: sync Grid Visualizer and Kotlin sample with Plasma/USDT schema changes#771claude[bot] wants to merge 1 commit into
Conversation
…changes Add PLASMA_WALLET support and USDT on Ethereum to keep documentation artifacts in sync with recent OpenAPI schema updates: - Grid Visualizer: add PLASMA_WALLET account type spec - Grid Visualizer: add ETHEREUM_WALLET and PLASMA_WALLET to USDT crypto asset - Kotlin sample: add PLASMA_WALLET handler to external account creation Related commits: - 927ce42 Add Plasma (PLASMA_WALLET) to Grid public API schema - 63b759d feat(payment-instructions): allow USDT as an Ethereum wallet assetType Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
| { type: 'ETHEREUM_WALLET', label: 'Wallet', network: 'Ethereum' }, | ||
| { type: 'PLASMA_WALLET', label: 'Wallet', network: 'Plasma' }, |
There was a problem hiding this comment.
Network options use missing icons
When the USDT network selector renders the new Ethereum and Plasma options, the dropdown derives /networks/ethereum.svg and /networks/plasma.svg, but those assets are absent, causing both options to display broken images.
Knowledge Base Used: Demo Apps and Samples
Prompt To Fix With AI
This is a comment left during a code review.
Path: components/grid-visualizer/src/data/crypto.ts
Line: 44-45
Comment:
**Network options use missing icons**
When the USDT network selector renders the new Ethereum and Plasma options, the dropdown derives `/networks/ethereum.svg` and `/networks/plasma.svg`, but those assets are absent, causing both options to display broken images.
**Knowledge Base Used:** [Demo Apps and Samples](https://app.greptile.com/lightspark/-/custom-context/knowledge-base/lightsparkdev/grid-api/-/docs/demo-apps-and-samples.md)
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
Greptile SummarySynchronizes Plasma and expanded USDT support across the Grid Visualizer and Kotlin sample.
Confidence Score: 4/5The missing Ethereum and Plasma network assets should be added before merging so the new USDT choices do not render broken images. The new options are reachable through the existing network dropdown, which derives URLs for two SVG assets that are not present in the visualizer. Files Needing Attention: components/grid-visualizer/src/data/crypto.ts and the corresponding public network assets
|
| Filename | Overview |
|---|---|
| components/grid-visualizer/src/data/account-types.ts | Adds a Plasma wallet account specification with the expected address field and no beneficiary. |
| components/grid-visualizer/src/data/crypto.ts | Adds Ethereum and Plasma USDT choices, but both resolve to network icon assets that are absent. |
| samples/kotlin/src/main/kotlin/com/grid/sample/routes/ExternalAccounts.kt | Adds a Plasma wallet handler consistent with sibling wallet builders, although compatibility with the externally pinned SDK could not be independently established. |
Prompt To Fix All With AI
### Issue 1
components/grid-visualizer/src/data/crypto.ts:44-45
**Network options use missing icons**
When the USDT network selector renders the new Ethereum and Plasma options, the dropdown derives `/networks/ethereum.svg` and `/networks/plasma.svg`, but those assets are absent, causing both options to display broken images.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (1): Last reviewed commit: "docs: sync Grid Visualizer and Kotlin sa..." | Re-trigger Greptile
Summary
Context
Recent OpenAPI schema changes added PLASMA_WALLET support and USDT on Ethereum. This PR syncs the documentation artifacts (Grid Visualizer, Kotlin sample) with these changes.
Related commits:
Other schema changes reviewed (no doc updates needed)
The following recent schema changes were reviewed and require no documentation updates:
Mintlify docs already include PLASMA_WALLET in
snippets/internal-accounts.mdx.Test plan
🤖 Generated with Claude Code