[Dashboard] Add thirdweb API option alongside Engine API #7734
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThe changes introduce a dual-mode API selection feature in the transactions explorer UI, allowing users to toggle between "thirdweb API" and "Engine API" modes. Components and code samples dynamically update according to the selected API. This includes UI controls, code example rendering, and API reference configuration, with corresponding prop and environment variable updates throughout the relevant components. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant TransactionsExplorerPage
participant TryItOut
participant Scalar
User->>TransactionsExplorerPage: Loads page
TransactionsExplorerPage->>User: Renders API mode dropdown
User->>TransactionsExplorerPage: Selects API mode ("thirdweb" or "engine")
TransactionsExplorerPage->>TryItOut: Passes useEngineAPI flag
TransactionsExplorerPage->>Scalar: Passes useEngineAPI flag
TryItOut->>User: Renders code examples for selected API
Scalar->>User: Renders API reference for selected API
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15–20 minutes Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (5)
🧰 Additional context used📓 Path-based instructions (3)**/*.{ts,tsx}📄 CodeRabbit Inference Engine (CLAUDE.md)
Files:
**/*.{ts,tsx,js,jsx}📄 CodeRabbit Inference Engine (CLAUDE.md)
Files:
apps/{dashboard,playground-web}/**/*.{ts,tsx}📄 CodeRabbit Inference Engine (CLAUDE.md)
Files:
🧠 Learnings (6)📓 Common learningsapps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/transactions/analytics/send-test-tx.client.tsx (11)Learnt from: arcoraven Learnt from: MananTank Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: jnsdls Learnt from: CR Learnt from: CR apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/transactions/explorer/page.tsx (12)Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: jnsdls Learnt from: CR Learnt from: CR apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/transactions/layout.tsx (23)Learnt from: MananTank Learnt from: CR Learnt from: Yash094 Learnt from: CR Learnt from: jnsdls Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: MananTank Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: MananTank Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: jnsdls Learnt from: jnsdls Learnt from: saminacodes Learnt from: MananTank Learnt from: CR Learnt from: MananTank apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/transactions/explorer/components/scalar.tsx (13)Learnt from: arcoraven Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: MananTank apps/dashboard/src/app/(app)/team/[team_slug]/[project_slug]/(sidebar)/transactions/server-wallets/components/try-it-out.tsx (24)Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: CR Learnt from: MananTank Learnt from: MananTank Learnt from: CR Learnt from: MananTank Learnt from: MananTank Learnt from: MananTank Learnt from: MananTank ⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
🔇 Additional comments (11)
✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7734 +/- ##
==========================================
- Coverage 56.22% 56.22% -0.01%
==========================================
Files 905 905
Lines 58880 58880
Branches 4135 4134 -1
==========================================
- Hits 33107 33106 -1
Misses 25669 25669
- Partials 104 105 +1
🚀 New features to boost your workflow:
|
size-limit report 📦
|
joaquim-verges
deleted the
_Dashboard_Add_thirdweb_API_option_alongside_Engine_API
branch
PR-Codex overview
This PR updates the transaction handling in the dashboard by introducing a selection between two API modes:
thirdwebandEngine API. It modifies components to accommodate this new functionality and updates API references accordingly.Detailed summary
<TryItOut />to acceptuseEngineAPIprop.NEXT_PUBLIC_ENGINE_CLOUD_URLwithNEXT_PUBLIC_THIRDWEB_API_HOSTin various components.TransactionsExplorerPage.<Scalar />to useuseEngineAPIprop.thirdwebandEngine API.Summary by CodeRabbit
New Features
Style