-
Notifications
You must be signed in to change notification settings - Fork 567
Dashboard: migrate contract/proposals from chakra to tailwind #7751
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
Dashboard: migrate contract/proposals from chakra to tailwind #7751
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
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
🪧 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 (
|
01aa5fa
to
f623d3e
Compare
6835c96
to
51c3d09
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7751 +/- ##
=======================================
Coverage 56.42% 56.42%
=======================================
Files 904 904
Lines 58665 58665
Branches 4135 4135
=======================================
Hits 33100 33100
Misses 25460 25460
Partials 105 105
🚀 New features to boost your workflow:
|
size-limit report 📦
|
Merge activity
|
f623d3e
to
90e3910
Compare
51c3d09
to
a757938
Compare
90e3910
to
650b0c3
Compare
a757938
to
00c95a2
Compare
<!-- ## 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 enhances the `DelegateButton` and `ProposalButton` components in a proposal management interface. It improves error handling, updates UI components, and modifies the structure of the `ContractProposalsPage` to enhance user experience and maintainability. ### Detailed summary - Added `parseError` utility for improved error handling. - Updated `DelegateButton` to show success/error messages using `toast`. - Refactored `ContractProposalsPage` to use a cleaner layout with improved proposal handling. - Enhanced `ProposalButton` with a new form structure and error messages. - Updated `Proposal` component to use new UI elements like `Badge` and `CardContent`. - Improved voting feedback by displaying vote counts and statuses more clearly. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
650b0c3
to
0361704
Compare
00c95a2
to
fad7cb9
Compare
PR-Codex overview
This PR enhances the proposal management functionality in the dashboard by improving user feedback on token delegation and proposal creation processes, while also updating UI components for better consistency and usability.
Detailed summary
parseError
utility for improved error handling.DelegateButton
to show success/error messages usingtoast
with detailed error descriptions.ContractProposalsPage
layout for improved visual hierarchy and added a message for no proposals found.ProposalButton
with better form handling usingForm
components and error messages.Proposal
component to useBadge
for proposal state representation and improved vote casting feedback with detailed error handling.