Fix: Universal Bridge Analytics Parsing and Team Analytics Loading State #7288
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
• Refactor analytics page into smaller async components with proper loading states • Add Promise.allSettled for better error handling in analytics data fetching • Fix date comparison logic using ISO string comparison for Universal Bridge metrics • Add infinity check for trend values to prevent display issues • Replace generic loading page with specific chart loading skeleton states 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
gregfromstl
requested review from
jnsdls,
joaquim-verges and
a team
as code owners
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Warning Rate limit exceeded@github-actions[bot] has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 45 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (84)
WalkthroughThis update introduces BridgeEmbed 2.0, a comprehensive cross-platform payment and bridge widget for React and React Native. It adds a modular state machine-driven UI, hooks, adapters, error handling, and Storybook stories for multi-step blockchain payment flows. The PR also refactors dashboard analytics, updates bridge/pay conversion logic, and enhances documentation and testing. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant BridgeEmbed
participant StateMachine
participant Hooks
participant Adapters
participant SDK
participant Wallet
User->>BridgeEmbed: Initiate payment/bridge flow
BridgeEmbed->>StateMachine: Initialize with mode and adapters
StateMachine->>Hooks: Fetch payment methods, routes, quotes
Hooks->>SDK: Request bridge/payment data
SDK-->>Hooks: Return data (routes, quotes, tokens)
Hooks-->>StateMachine: Provide payment methods/quotes
StateMachine->>BridgeEmbed: Update UI state (selection, preview, execution)
User->>BridgeEmbed: Select payment method, confirm details
BridgeEmbed->>StateMachine: Dispatch events (selection, confirm, execute)
StateMachine->>Hooks: Prepare and execute steps
Hooks->>Wallet: Send transactions, sign, switch chain
Wallet-->>Hooks: Return transaction status/receipts
Hooks-->>StateMachine: Report execution progress/completion/errors
StateMachine->>BridgeEmbed: Update UI (progress, success, error)
User->>BridgeEmbed: Retry/cancel as needed
Possibly related PRs
Suggested reviewers
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 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 (
|
github-actions
bot
added
Dashboard
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. |
size-limit report 📦
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 introduces several enhancements and new features across various components in the
thirdwebpackage, including type definitions, UI adjustments, and error handling improvements.Detailed summary
Actiontype inBridgeAction.ts.modalMaxWidthCompactinconstants.ts.getQueryOptionsinuseChainQuery.ts.WindowAdapterinterface for platform-specific functionality.mapBridgeError.ts.useBridgeRoutesanduseBridgeErrorhooks.FiatProviderSelectioncomponent for selecting payment providers.TokenBalanceRow,ErrorBanner, andSuccessScreen.WalletRow.Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Style
Tests
Chores