Skip to content

Commit

Permalink
chore: update code based on comments
Browse files Browse the repository at this point in the history
  • Loading branch information
suisin-deriv committed May 2, 2024
1 parent 1331686 commit 6cdfaf5
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ export const PersonalDetailsForm = observer(({ history }: { history: BrowserHist
id={'phone'}
label={localize('Phone number*')}
className={
account_settings.phone_number_verification.verified
account_settings?.phone_number_verification?.verified
? 'account-form__fieldset--phone'
: ''
}
Expand All @@ -410,7 +410,7 @@ export const PersonalDetailsForm = observer(({ history }: { history: BrowserHist
error={errors.phone}
disabled={
isFieldDisabled('phone') ||
account_settings.phone_number_verification.verified
account_settings?.phone_number_verification?.verified
}
data-testid='dt_phone'
/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export const VerifyButton = observer(() => {
const { is_mobile } = ui;
const { account_settings } = client;
const { phone_number_verification } = account_settings;
const { verified } = phone_number_verification;
const { verified: phone_number_verified } = phone_number_verification;
const history = useHistory();

const redirectToPhoneVerification = () => {
Expand All @@ -22,7 +22,7 @@ export const VerifyButton = observer(() => {

return (
<div className='account-form__phone-verification-btn'>
{verified ? (
{phone_number_verified ? (
<div className='account-form__phone-verification-btn--verified'>
<LegacyWonIcon iconSize='xs' />
<CaptionText bold color='#4bb4b3'>
Expand Down
4 changes: 2 additions & 2 deletions packages/account/src/Styles/account.scss
Original file line number Diff line number Diff line change
Expand Up @@ -448,10 +448,10 @@ $MIN_HEIGHT_FLOATING: calc(

.phone-verification__popover {
position: absolute;
right: -3rem;
inset-inline-end: -3rem;

@include mobile {
right: 0rem;
inset-inline-end: 0rem;
}
}
}
Expand Down

0 comments on commit 6cdfaf5

Please sign in to comment.