Closed Yh010 closed 2 months ago
✅ All Clear: This PR is ready to merge! 👍
✨ Generated with love by Kaizen ❤️
Useful Commands
Feedback: Reply with !feedback [your message]
Ask PR: Reply with !ask-pr [your question]
Review: Reply with !review
@CodeTorso @Dhravya this is a fresh PR in reference to #155 .
Are you sure this fixes it though?
Fixes Multiple Memory Submission Bug
Overview
This pull request addresses issue #146 by implementing a solution to prevent multiple submissions of a form, which was causing memory-related bugs. The main change involves utilizing the
useFormStatus
hook to manage the state of the submit button, ensuring it is disabled during form submission.Changes
Key Changes:
SubmitButton
component that leveragesuseFormStatus
to disable the button when a submission is in progress.DialogFooter
and replaced it with the newSubmitButton
component for better encapsulation and readability.New Features:
SubmitButton
component provides a more user-friendly experience by preventing multiple submissions, thereby enhancing form reliability.Refactoring:
SubmitButton
component, improving code organization and maintainability.Menu
component cleaner and easier to understand.Original Description
used useFormStatus to disable to submit button