Closed Kodylow closed 4 months ago
apps/guardian-ui/src/components/SetConfiguration.tsx (5)
`15-22`: Imports for modal components and `useDisclosure` have been added. These imports are necessary for the new modal functionality to confirm the password. Ensure that these components are properly utilized in the component's logic. --- `73-73`: Introduced a new state variable `confirmPassword`. This state is essential for storing the user's input to confirm the password. It's a good practice to manage such state locally in the component where it's used. --- `92-92`: `useDisclosure` hook is used to manage the modal's open/close state. This is a standard approach with Chakra UI for controlling visibility states of modals. Good use of React hooks for UI state management. --- `172-178`: Updated `handleNext` function to include password confirmation logic. This update is crucial for the new feature to ensure that the password and confirmPassword match before proceeding. This is a good security practice. --- `446-488`: Added a modal component for password confirmation. The modal setup is correctly implemented using Chakra UI components. It includes all necessary parts like overlay, content, header, body, and footer. The logic to handle password mismatch and proceed on match is correctly placed.
reopen of https://github.com/fedimint/ui/pull/423
adds a confirm password modal before continuing during setup
Summary by CodeRabbit
New Features
Enhancements
Dependencies
husky
dependency from version^8.0.3
to^9.0.11
.