-
Notifications
You must be signed in to change notification settings - Fork 603
[MNY-240] Dashboard: Show bridge status info on transaction page if available #8241
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
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 #8241 +/- ##
=======================================
Coverage 55.02% 55.02%
=======================================
Files 919 919
Lines 60583 60583
Branches 4127 4127
=======================================
Hits 33335 33335
Misses 27145 27145
Partials 103 103
🚀 New features to boost your workflow:
|
size-limit report 📦
|
Merge activity
|
…vailable (#8241) <!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces a new tabbed interface for bridge status and overview details in the transaction view. It enhances user experience by allowing easy navigation between different states of a bridge transaction. ### Detailed summary - Added `BridgeAndOverviewTabs` component for tab navigation. - Implemented `BridgeStatus` component to display bridge transaction status. - Created stories for different bridge statuses (`Completed`, `Pending`, `Failed`, `NotFound`). - Enhanced `Page` component to include bridge status with polling. - Refactored transaction details layout for better clarity. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
458097d
to
85a6afc
Compare
e3c4326
to
cccbde9
Compare
PR-Codex overview
This PR introduces a new component structure for displaying bridge transaction statuses and an overview in a tabbed interface, enhancing user experience on the dashboard. It also adds storybook stories for various bridge statuses and integrates polling for real-time updates.
Detailed summary
BridgeAndOverviewTabs
component for tabbed navigation between bridge status and overview.BridgeStatus
component to display detailed bridge transaction information.BridgeStatusWithPolling
for real-time status updates.Page
component to include the new tab structure and bridge status.