-
Notifications
You must be signed in to change notification settings - Fork 327
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
Migrate Network Config Modal from chakra to shadcn/tailwind #4745
Migrate Network Config Modal from chakra to shadcn/tailwind #4745
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 Graphite |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4745 +/- ##
=======================================
Coverage 48.92% 48.92%
=======================================
Files 1050 1050
Lines 57002 57002
Branches 3892 3892
=======================================
Hits 27889 27889
Misses 28480 28480
Partials 633 633
*This pull request uses carry forward flags. Click here to find out more. |
size-limit report 📦
|
apps/dashboard/src/components/configure-networks/ConfigureNetworkForm.tsx
Outdated
Show resolved
Hide resolved
ec520ca
to
53a9505
Compare
2e04236
to
2c8d52d
Compare
53a9505
to
c4e7e01
Compare
c4e7e01
to
c042a12
Compare
Merge activity
|
c042a12
to
2856ffc
Compare
## Problem solved Short description of the bug fixed or feature added <!-- start pr-codex --> --- ## PR-Codex overview This PR removes unused components, updates UI elements, and refactors form fields in the dashboard application. ### Detailed summary - Deleted `TooltipBox.tsx` and `ConfirmationPopover.tsx` - Updated UI elements in `IconUpload.tsx` and `ConfigureNetworkModal.tsx` - Refactored form fields in `ChainIdInput.tsx`, `RpcInput.tsx`, and `NetworkIDInput.tsx` > The following files were skipped due to too many changes: `apps/dashboard/src/@/components/ui/radio-group.tsx`, `apps/dashboard/src/components/configure-networks/ConfigureNetworkForm.tsx` > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
2856ffc
to
bc72883
Compare
Problem solved
Short description of the bug fixed or feature added
PR-Codex overview
This PR removes deprecated components, updates UI elements, and refactors form fields in the dashboard.
Detailed summary
TooltipBox.tsx
,ConfirmationPopover.tsx
,thirdweb.client.ts
ChainIdInput.tsx
,RpcInput.tsx
,NetworkIdInput.tsx
, andFormFieldSetup.tsx