Skip to content

Commit

Permalink
fix(wallets): 🚑 allow adding an account with common password
Browse files Browse the repository at this point in the history
  • Loading branch information
heorhi-deriv committed Oct 31, 2024
1 parent 792d672 commit 83f9987
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,15 @@ import PasswordMeter from './PasswordMeter';
import PasswordViewerIcon from './PasswordViewerIcon';
import './WalletPasswordField.scss';

export const validatePassword = (
password: string,
mt5Policy: boolean,
localize: ReturnType<typeof useTranslations>['localize']
) => {
type TValidatePasswordProps = {
isMT5PasswordNotSet?: boolean;
localize: ReturnType<typeof useTranslations>['localize'];
mt5Policy: boolean;
password: string;
};

export const validatePassword = (values: TValidatePasswordProps) => {
const { isMT5PasswordNotSet, localize, mt5Policy, password } = values;
const score = mt5Policy ? calculateScoreMT5(password) : calculateScoreCFD(password);
let validationErrorMessage = '';

Expand All @@ -38,7 +42,9 @@ export const validatePassword = (
} else {
cfdSchema(localize).validateSync(password);
}
validationErrorMessage = getWarningMessages(localize)[feedback.warning as passwordKeys] ?? '';
validationErrorMessage = isMT5PasswordNotSet
? getWarningMessages(localize)[feedback.warning as passwordKeys] ?? ''
: '';
} catch (err) {
if (err instanceof ValidationError) {
validationErrorMessage = err?.message;
Expand All @@ -50,6 +56,7 @@ export const validatePassword = (

const WalletPasswordField: React.FC<WalletPasswordFieldProps> = ({
autoComplete,
isMT5PasswordNotSet,
label,
mt5Policy = false,
name = 'walletPasswordField',
Expand All @@ -67,8 +74,8 @@ const WalletPasswordField: React.FC<WalletPasswordFieldProps> = ({
const [errorMessage, setErrorMessage] = useState('');

const { score, validationErrorMessage } = useMemo(
() => validatePassword(password, mt5Policy, localize),
[password, mt5Policy, localize]
() => validatePassword({ password, mt5Policy, localize, isMT5PasswordNotSet }),

Check failure on line 77 in packages/wallets/src/components/Base/WalletPasswordField/WalletPasswordField.tsx

View workflow job for this annotation

GitHub Actions / Build And Test

Expected object keys to be in natural insensitive ascending order. 'mt5Policy' should be before 'password'

Check failure on line 77 in packages/wallets/src/components/Base/WalletPasswordField/WalletPasswordField.tsx

View workflow job for this annotation

GitHub Actions / Build And Test

Expected object keys to be in natural insensitive ascending order. 'localize' should be before 'mt5Policy'

Check failure on line 77 in packages/wallets/src/components/Base/WalletPasswordField/WalletPasswordField.tsx

View workflow job for this annotation

GitHub Actions / Build And Test

Expected object keys to be in natural insensitive ascending order. 'isMT5PasswordNotSet' should be before 'localize'
[password, mt5Policy, localize, isMT5PasswordNotSet]
);
const passwordValidation = mt5Policy ? !validPasswordMT5(password) : !validPassword(password);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import { Loader } from '@deriv-com/ui';
import { WalletTextFieldProps } from '../WalletTextField/WalletTextField';

export interface WalletPasswordFieldProps extends WalletTextFieldProps {
isMT5PasswordNotSet?: boolean;
mt5Policy?: boolean; // This prop is used to utilize the new password validation for MT5.
password: string;
passwordError?: boolean;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -292,6 +292,7 @@ const MT5PasswordModal: React.FC<TProps> = ({ isVirtual, marketType, platform, p
return (
<EnterPassword
isLoading={tradingPlatformPasswordChangeLoading || createMT5AccountLoading}
isMT5PasswordNotSet={isMT5PasswordNotSet}
isTncChecked={isTncChecked}
isVirtual={activeWalletData?.is_virtual}
marketType={marketType}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,15 @@ import { Localize, useTranslations } from '@deriv-com/translations';
import { Button, Text, useDevice } from '@deriv-com/ui';
import { WalletPasswordFieldLazy } from '../../../../components/Base';
import { THooks, TMarketTypes, TPlatforms } from '../../../../types';
import { validPassword, validPasswordMT5 } from '../../../../utils/password-validation';
import { validPassword } from '../../../../utils/password-validation';
import { CFDPasswordModalTnc } from '../../components/CFDPasswordModalTnc';
import { CFD_PLATFORMS, getMarketTypeDetails, PlatformDetails, PRODUCT } from '../../constants';
import './EnterPassword.scss';

type TProps = {
isForgotPasswordLoading?: boolean;
isLoading?: boolean;
isMT5PasswordNotSet?: boolean;
isTncChecked?: boolean;
isVirtual?: boolean;
marketType: TMarketTypes.CreateOtherCFDAccount;
Expand All @@ -30,6 +31,7 @@ type TProps = {
const EnterPassword: React.FC<TProps> = ({
isForgotPasswordLoading,
isLoading,
isMT5PasswordNotSet,
isTncChecked = true,
isVirtual,
marketType,
Expand All @@ -48,8 +50,6 @@ const EnterPassword: React.FC<TProps> = ({
const { localize } = useTranslations();
const { data } = useActiveWalletAccount();

const isMT5 = platform === CFD_PLATFORMS.MT5;
const disableButton = isMT5 ? !validPasswordMT5(password) : !validPassword(password);
const accountType = data?.is_virtual ? localize('Demo') : localize('Real');
const title = PlatformDetails[platform].title;
const marketTypeTitle =
Expand All @@ -60,6 +60,7 @@ const EnterPassword: React.FC<TProps> = ({
'Hint: You may have entered your Deriv password, which is different from your {{title}} password.',
{ title }
);
const isAddAccountBtnDisabled = !password || isLoading || !validPassword(password) || !isTncChecked;

useEffect(() => {
if (passwordError) {
Expand Down Expand Up @@ -89,6 +90,7 @@ const EnterPassword: React.FC<TProps> = ({
/>
</Text>
<WalletPasswordFieldLazy
isMT5PasswordNotSet={isMT5PasswordNotSet}
label={localize('{{title}} password', { title })}
onChange={onPasswordChange}
password={password}
Expand Down Expand Up @@ -122,7 +124,7 @@ const EnterPassword: React.FC<TProps> = ({
<Localize i18n_default_text='Forgot password?' />
</Button>
<Button
disabled={!password || isLoading || disableButton || !isTncChecked}
disabled={isAddAccountBtnDisabled}
isLoading={isLoading}
onClick={onPrimaryClick}
size='lg'
Expand Down

0 comments on commit 83f9987

Please sign in to comment.