-
Notifications
You must be signed in to change notification settings - Fork 560
[Dashboard] Support in dashboard #7596
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 Git ↗︎
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 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. |
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #7596 +/- ##
==========================================
- Coverage 56.46% 56.45% -0.01%
==========================================
Files 906 906
Lines 58110 58115 +5
Branches 4222 4226 +4
==========================================
Hits 32810 32810
- Misses 25190 25195 +5
Partials 110 110
🚀 New features to boost your workflow:
|
size-limit report 📦
|
…s a fraction (#7597) <!-- ## 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 focuses on updating the `SupplyClaimedProgress` component to use `claimedSupplyTokens` and `totalSupplyTokens` as `number` types instead of `bigint`, along with minor adjustments in related components and UI elements. ### Detailed summary - Changed `enable-background` to `enableBackground` in `DiscordIcon.tsx`. - Updated `SupplyClaimedProgress` props from `claimedSupply` and `totalSupply` (bigint) to `claimedSupplyTokens` and `totalSupplyTokens` (number). - Adjusted calculations and conditions in `SupplyClaimedProgress` for new props. - Modified test cases in stories to reflect new prop types. - Updated UI elements to display supply values correctly based on the new number type. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Updated supply progress components and related usage to use number types instead of bigints, with new prop names for clarity. * Improved handling and display of "unlimited" supply scenarios in supply progress components. * Adjusted percentage calculation and formatting for claimed supply progress. * **Style** * Corrected SVG attribute naming in the Discord icon to ensure proper rendering. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
<!-- ## 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 focuses on refining the UI components and enhancing the functionality of the `ERC20` public page. It includes improvements in layout, styling, and the introduction of new features related to token price data and analytics. ### Detailed summary - Deleted unused files: `useTokenPriceData.ts` and `ContractCreatorBadge.tsx`. - Updated `PageHeader` to change border style. - Added `tooltip` prop in `CopyAddressButton`. - Changed API key usage in `fetchTokenInfoFromBridge`. - Enhanced `ThirdwebAreaChart` with better props handling. - Introduced `GridPattern` component for background. - Improved `RecentTransfers` UI and pagination buttons. - Updated `PriceChart` and `TokenStats` to directly use passed props for token price data. - Refactored `ContractHeaderUI` to improve layout and social link handling. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Introduced a customizable grid background pattern component for enhanced visual design. * Added support for displaying token price data and analytics on ERC20 public pages, including conditional rendering of token statistics. * Enabled tooltip customization for the copy address button. * **Bug Fixes** * The contract creator badge icon is now visible, and spacing between elements has been improved. * **Refactor** * Simplified and unified chart headers and analytics section layouts. * Streamlined fetching and handling of token price data with a new asynchronous function and removed the previous hook. * Improved layout and container styling across ERC20 and NFT public pages, including card backgrounds, borders, and spacing. * Adjusted component structures for better visual separation and consistency. * Removed the contract creator badge component and replaced it with a simpler contract owner link. * **Style** * Updated card and header styles for better visual hierarchy and clarity. * Enhanced recent transfers and contract header sections with improved padding, borders, and rounded corners. * Removed small shadow effects from cards and refined header background and border styles. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Cursor Agent <[email protected]>
PR-Codex overview
This PR focuses on updating various components and functionalities within the codebase, including version updates, refactoring for better code organization, and UI improvements. It also introduces new support forms and adjusts existing ones to enhance user experience.
Detailed summary
version
inpackage.json
files forthirdweb
,nebula
, andwagmi-adapter
.SupplyClaimedProgress
to useclaimedSupplyTokens
andtotalSupplyTokens
as numbers.ContractHeaderUI
and various card components.framer-rewrites.js
to replace/universal-bridge
with/payments
.SupportAIChatCard
with a chat interface for user support.GridPattern
in background patterns for better styling.