-
Notifications
You must be signed in to change notification settings - Fork 525
Dashboard: Fix team invite UI not showing feedback on submit #6866
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: Fix team invite UI not showing feedback on submit #6866
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
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. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6866 +/- ##
=======================================
Coverage 55.31% 55.31%
=======================================
Files 896 896
Lines 57024 57024
Branches 3968 3968
=======================================
Hits 31542 31542
Misses 25385 25385
Partials 97 97
🚀 New features to boost your workflow:
|
2b39c5d
to
a45e91d
Compare
size-limit report 📦
|
Merge activity
|
<!-- ## 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 improving the `InviteSection` component by modifying the form handling logic and enhancing the user interface with additional functionality. ### Detailed summary - Removed the `useEffect` hook that reset the mutation result on form updates. - Changed the form's `onChange` to reset the mutation result if `sendInvites.data` exists. - Updated the button layout to include a `Reset` button for clearing the form and resetting invites. - Adjusted class names for better spacing and alignment. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
a45e91d
to
692c25e
Compare
PR-Codex overview
This PR focuses on refactoring the
InviteSection
component in theInviteSection.tsx
file. It improves the handling of form updates and adds a reset button for the form.Detailed summary
useEffect
hook for resetting mutation results.onChange
handler to the form that resets the mutation result when form updates occur.Reset
button to reset the form and mutation state.