devin-ai-integration[bot] commented 2 weeks ago

Convert app/components/UI/BlockingActionModal/index.js to TypeScript

Changes Made

Renamed index.js to index.tsx
Removed PropTypes import and prop type definitions
Defined BlockingActionModalProps interface for component props
Added type annotations for function parameters and variables
Imported Theme type from '../../../util/theme/models'
Added type for colors parameter in createStyles function
Made onAnimationCompleted prop optional to maintain compatibility
Reverted changes to ConnectQRHardware/index.tsx

Motivation

This change is part of our ongoing effort to migrate the codebase to TypeScript, improving type safety and developer experience.

Context

The BlockingActionModal component is used in various parts of the application to display a modal with a loading spinner. Converting it to TypeScript will help catch potential type-related errors and improve code maintainability.

Related Issues

N/A

Manual Testing Steps

Verify that the BlockingActionModal component still functions correctly in all places it's used.
Check that the loading spinner appears when isLoadingAction is true.
Ensure that the modal shows and hides properly based on the modalVisible prop.

Notes

The onAnimationCompleted prop was made optional to avoid unnecessary modifications to other files using this component.
No changes were required in the component's logic or styling.
Changes to ConnectQRHardware/index.tsx have been reverted to ensure this PR only contains intended changes related to the BlockingActionModal component.
The scope of this PR is limited to the TypeScript conversion of BlockingActionModal.

This Devin run was requested by Sampriti