-
Notifications
You must be signed in to change notification settings - Fork 619
Move Deployed Contracts Page to App Router #4995
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
Move Deployed Contracts Page to App Router #4995
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @MananTank and the rest of your teammates on |
7b82b42 to
76ecba5
Compare
76ecba5 to
be8d45f
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4995 +/- ##
=======================================
Coverage 47.77% 47.77%
=======================================
Files 1058 1058
Lines 57276 57276
Branches 3946 3946
=======================================
Hits 27362 27362
Misses 29223 29223
Partials 691 691
*This pull request uses carry forward flags. Click here to find out more. |
size-limit report 📦
|
be8d45f to
a4ee55c
Compare
a4ee55c to
2d4f96c
Compare
be1b851 to
bcf1624
Compare
|
@jnsdls Wrapped the contracts that fetches the contracts with suspense - you should see the shell of the page instantly now |
apps/dashboard/src/app/(dashboard)/dashboard/contracts/deploy/page.tsx
Outdated
Show resolved
Hide resolved
bcf1624 to
4b31754
Compare
Merge activity
|
## Problem solved
Short description of the bug fixed or feature added
<!-- start pr-codex -->
---
## PR-Codex overview
This PR focuses on enhancing the contract deployment and management features in the dashboard application. It introduces new components, improves layouts, and refines data handling for better user experience.
### Detailed summary
- Updated layout in `deploy.tsx` and `layout.tsx` to use flexbox for better responsiveness.
- Added `getAuthTokenWalletAddress` function in `getAuthToken.ts` for wallet address retrieval.
- Introduced `DeployedContractsPage` and `DeployedContractsTable` components for contract management.
- Refactored `DeployedContracts` component to accept contract list directly and handle loading state.
- Added `GetStartedWithContractsDeploy` component for user onboarding.
- Enhanced `DeployedContractsPageHeader` with import and deploy contract buttons.
- Improved sorting logic in `getSortedDeployedContracts.tsx`.
- Removed unused imports and cleaned up code for better readability.
> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`
<!-- end pr-codex -->
4b31754 to
92fbb18
Compare

Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR focuses on enhancing the dashboard and contract management features by improving layout components, adding new functionalities for contract handling, and refining the UI components for better user experience.
Detailed summary
deploy.tsxandlayout.tsxto use flexbox.getAuthTokenWalletAddressfunction for account management.DeployedContractsPageandDeployedContractsTablecomponents.GetStartedWithContractsDeploywith wallet connection logic.DeployedContractsto acceptcontractListandisPendingprops.ImportModaland button functionalities for contract management.