-
Notifications
You must be signed in to change notification settings - Fork 567
Dashboard: Migrate contract/permissions page from chakra to tailwind, UI improvements #7750
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/permissions page from chakra to tailwind, UI improvements #7750
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 (
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7750 +/- ##
=======================================
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 📦
|
01aa5fa
to
f623d3e
Compare
Merge activity
|
0d2be61
to
adcd69c
Compare
f623d3e
to
90e3910
Compare
90e3910
to
650b0c3
Compare
adcd69c
to
345eb3d
Compare
… UI improvements (#7750) <!-- ## 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 enhancing the `Permissions` component and related files by improving UI elements, restructuring code for clarity, and integrating better user feedback mechanisms such as alerts and toast notifications. ### Detailed summary - Updated the `Permissions` component's return structure to include headings and descriptions. - Refined the `alertVariants` in `alert.tsx` for better styling. - Replaced `ButtonGroup` with a `div` for layout adjustments in the `Permissions` component. - Enhanced error handling with toast notifications in `permissions-editor.tsx`. - Improved form handling and cleaned up values before submission. - Replaced `DelayedDisplay` with an `Alert` component in `permissions-editor.tsx`. - Updated the `ContractPermission` component to utilize `Select` for role management instead of traditional dropdowns. - Added conditional rendering for alerts based on role states in `ContractPermission`. - Adjusted button styles and behavior for better user experience across components. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
345eb3d
to
04c7243
Compare
650b0c3
to
0361704
Compare
PR-Codex overview
This PR focuses on enhancing the
Permissions
component and its related functionalities. It improves the user interface for managing permissions, updates the alert components, and refines the form handling logic.Detailed summary
Permissions
component to include a header and description.alert
component for consistency.permissions-editor
andcontract-permission
.