Add Solana transactions and wallets tables #8263
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: 8604340 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
|
Caution Review failedThe pull request is closed. WalkthroughAdds Solana support across dashboard transactions: unified EVM/Solana transaction tables, Solana transaction details UI, Solana test-transaction flow, Solana server wallets listing/creation via vault-sdk, and permissions upgrade flow. Removes legacy tables. Updates pages to fetch/render Solana data, introduces utilities/types, and adjusts sidebar labels. Vault SDK response shape renamed. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor U as User
participant D as Dashboard (FTUX)
participant M as SendTestSolanaTransaction Modal
participant P as engineCloudProxy
participant E as Engine Cloud
participant S as Solana Network
U->>D: Open "Try it out"
D->>M: Open modal (wallets, network)
U->>M: Select wallet, enter key/token, submit
M->>P: POST /v1/solana/transaction\nheaders: x-secret-key OR x-vault-access-token
P->>E: Forward request
E-->>S: Submit tx (base64 instruction)
S-->>E: Signature / status
E-->>P: 200 OK { signature, status }
P-->>M: Success
M->>D: Toast + close dialog
alt Error
E-->>P: Error (4xx/5xx)
P-->>M: Error
M->>U: Show error toast
end
sequenceDiagram
autonumber
actor U as User
participant T as UnifiedTransactionsTable
participant Q as useQuery (EVM)
participant QS as useQuery (Solana)
participant P as engineCloudProxy
U->>T: Toggle chain (evm/solana), set filters
alt EVM selected
T->>Q: Fetch EVM tx (status/id/from/page)
Q->>P: GET /v1/transactions
P-->>Q: Data or empty on error
Q-->>T: Render EVM table rows
else Solana selected
T->>QS: Fetch Solana tx (status/id/from/page)
QS->>P: GET /v1/solana/transactions
P-->>QS: Data or empty on error
QS-->>T: Render Solana table rows
end
loop Auto-update (if enabled)
T->>Q: Refetch (active chain)
end
sequenceDiagram
autonumber
participant Page as /transactions/tx/[id]/page.tsx
participant AE as getEVMTransaction + logs
participant AS as getSingleSolanaTransaction + logs
participant UIE as TransactionDetailsUI (EVM)
participant UIS as SolanaTransactionDetailsUI
Page->>AS: Fetch Solana tx by id
Page->>AE: Fetch EVM tx by id
alt Solana found
Page->>AS: Fetch Solana activity logs
Page->>UIS: Render Solana details
else EVM found
Page->>AE: Fetch EVM activity logs + decode
Page->>UIE: Render EVM details
else None
Page->>Page: notFound()
end
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes ✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (28)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8263 +/- ##
=======================================
Coverage 55.01% 55.01%
=======================================
Files 919 919
Lines 60588 60588
Branches 4126 4126
=======================================
Hits 33334 33334
Misses 27150 27150
Partials 104 104
🚀 New features to boost your workflow:
|
Merged
size-limit report 📦
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR-Codex overview
This PR focuses on enhancing Solana wallet and transaction functionalities within the application, including the addition of new types, components, and utility functions to support Solana transactions and wallet management.
Detailed summary
GetSolanaAccountsDatatype to useitemsinstead ofaccounts.SolanaWallettype definition.statusDetailsfor transaction statuses.Summary by CodeRabbit