Skip to content

Commit

Permalink
[WALL] Aizad/Fix Import Issues in Wallets (binary-com#15495)
Browse files Browse the repository at this point in the history
* fix: remove unused imports and add eslint ignore to useeffect

* chore: replace modal.isOpen with modal

* chore: update useEffect dependencies
  • Loading branch information
aizad-deriv committed Jun 6, 2024
1 parent 2572093 commit f762463
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import React from 'react';
import { ModalStepWrapper } from '../../../../src/components/Base/ModalStepWrapper';
import { FlowProvider, TFlowProviderContext, useFlow } from '../../../../src/components/FlowProvider';
import { useModal } from '../../../../src/components/ModalProvider';
import { MT5AccountType } from '../../../../src/features/cfd/screens/MT5AccountType';
import VerificationFlow from './VerificationFlow';

const PasswordScreen = () => {
Expand Down Expand Up @@ -62,7 +61,6 @@ const screens = {
bScreen: <ScreenB />,
JurisdictionScreen: <JurisdictionScreen />,
passwordScreen: <PasswordScreen />,
selectAccountTypeScreen: <MT5AccountType onMarketTypeSelect={marketType => marketType} selectedMarketType='all' />,
};

const AccountFlow = () => {
Expand Down Expand Up @@ -91,7 +89,7 @@ const AccountFlow = () => {
testb: '',
}}
screens={screens}
screensOrder={['selectAccountTypeScreen', 'JurisdictionScreen', 'passwordScreen', 'aScreen', 'bScreen']}
screensOrder={['JurisdictionScreen', 'passwordScreen', 'aScreen', 'bScreen']}
>
{context => {
return (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ const WalletTourGuide = () => {
if (needToStart && modal.isOpen) {
modal.hide();
}
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [needToStart, modal.isOpen]);

useEffect(() => {
Expand Down

0 comments on commit f762463

Please sign in to comment.