Skip to content

Commit 205389c

Browse files
tomasjakljbazant
tomasjakl
authored and
jbazant
committed
feat(suite-native): update copy to final in mobile swap
1 parent 1f02cbb commit 205389c

File tree

12 files changed

+32
-31
lines changed

12 files changed

+32
-31
lines changed

‎suite-native/intl/src/messages.ts‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ export const messages = {
329329
description:
330330
'<type></type> is the default address type. <moreLink>Learn more</moreLink>',
331331
buttons: {
332-
select: 'Change account type',
332+
select: 'Change address type',
333333
confirm: 'Continue with <type></type>',
334334
},
335335
},
@@ -2246,12 +2246,12 @@ export const messages = {
22462246
},
22472247
selectCoinToSell: {
22482248
title: 'You pay',
2249-
buttonTitle: 'Select coin',
2249+
buttonTitle: 'Select asset',
22502250
amountLabel: 'You pay',
22512251
},
22522252
selectCoin: {
22532253
title: 'You get',
2254-
buttonTitle: 'Select coin',
2254+
buttonTitle: 'Select asset',
22552255
amountLabel: 'You get',
22562256
},
22572257
selectRate: {
@@ -2404,7 +2404,8 @@ export const messages = {
24042404
},
24052405
tradingExchangeApprovalScreen: {
24062406
title: 'Set {symbol} spending',
2407-
subtitle: 'Set how much {symbol} to approve for this swap.',
2407+
subtitle:
2408+
'Set the {symbol} amount to approve so {companyName} can access it. This is required to continue with your swap.',
24082409
revokeSuccessAlert: 'Revoke successful. Set a higher limit.',
24092410
lowLimitInfoAlert:
24102411
'You’ve approved this token, but the limit is too low. Increase it to continue.',
@@ -2431,7 +2432,7 @@ export const messages = {
24312432
title: 'Fee picker',
24322433
description: {
24332434
title: 'Transaction fee',
2434-
body: 'Fees are paid directly to network miners for processing your transactions.',
2435+
body: 'Fees are paid directly to validators for processing your transactions.',
24352436
},
24362437
},
24372438
tradingReviewOutputs: {
@@ -2442,11 +2443,11 @@ export const messages = {
24422443
title: 'Set limit',
24432444
unlimitedCard: {
24442445
description:
2445-
"Skip future approval requests and save on fees by approving unlimited {symbol}. {companyName} will have full access to your {symbol}, and your funds may be at risk if they're ever compromised.",
2446+
'Approve unlimited {symbol} to skip future approval requests and reduce fees. Only use this option if you trust {companyName}, as it will have access to all your {symbol}.',
24462447
},
24472448
limitedCard: {
24482449
description:
2449-
"Approve only the amount needed for this swap. You'll need to approve again (and pay a fee) for future swaps, but this reduces risk by keeping you in full control of your {symbol}.",
2450+
"Approve only the amount needed for this swap. This helps reduce risk, but you'll need to approve again (and pay a fee) for future swaps.",
24502451
},
24512452
},
24522453
exchangeTradePreviewCard: {

‎suite-native/module-trading/src/components/buy/__tests__/BuyForm.comp.test.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('BuyForm', () => {
3131
const { queryByText, getByText, getByLabelText } = await renderBuyForm({}, result.current);
3232

3333
expect(getByText('You pay')).toBeTruthy();
34-
expect(getByLabelText('Select coin')).toHaveTextContent(/Select coin/);
34+
expect(getByLabelText('Select asset')).toHaveTextContent(/Select asset/);
3535
expect(queryByText('Receive account')).toBeNull();
3636
expect(queryByText('Payment method')).toBeNull();
3737
expect(queryByText('Country of residence')).toBeTruthy();
@@ -59,7 +59,7 @@ describe('BuyForm', () => {
5959
expect(getByText('You pay')).toBeTruthy();
6060

6161
expect(getByLabelText('Select fiat currency')).toHaveTextContent(/CZK/);
62-
expect(getByLabelText('Select coin')).toHaveTextContent(/Select coin/);
62+
expect(getByLabelText('Select asset')).toHaveTextContent(/Select asset/);
6363

6464
expect(queryByText('Receive account')).toBeNull();
6565

‎suite-native/module-trading/src/components/buy/__tests__/BuyTradeableAssetPicker.comp.test.tsx‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,10 @@ describe('BuyTradeableAssetPicker', () => {
4646
form = await renderFormHook();
4747
});
4848

49-
it('should render "Select coin" button with caret', async () => {
49+
it('should render "Select asset" button with caret', async () => {
5050
const { getByLabelText } = await renderTradeableAssetPicker();
5151

52-
expect(getByLabelText('Select coin')).toHaveTextContent(/^Select coin.$/);
52+
expect(getByLabelText('Select asset')).toHaveTextContent(/^Select asset.$/);
5353
});
5454

5555
it('should render bottom sheet with all assets', async () => {
@@ -69,7 +69,7 @@ describe('BuyTradeableAssetPicker', () => {
6969
it('should preselect BTC and do not render caret', async () => {
7070
const { getByLabelText } = await renderTradeableAssetPicker();
7171

72-
expect(getByLabelText('Select coin')).toHaveTextContent('BTC');
72+
expect(getByLabelText('Select asset')).toHaveTextContent('BTC');
7373
});
7474

7575
it('should not render bottom sheet at all', async () => {
@@ -82,10 +82,10 @@ describe('BuyTradeableAssetPicker', () => {
8282
const { getByLabelText } = await renderTradeableAssetPicker();
8383

8484
// no need to act as there should be no action
85-
fireEvent.press(getByLabelText('Select coin'));
85+
fireEvent.press(getByLabelText('Select asset'));
8686
fireEvent.press(getByLabelText('You get'));
8787

88-
expect(getByLabelText('Select coin')).toHaveTextContent('BTC');
88+
expect(getByLabelText('Select asset')).toHaveTextContent('BTC');
8989
});
9090
});
9191
});

‎suite-native/module-trading/src/components/exchange/ExchangeApprovalLimitSheet/__tests__/ExchangeApprovalLimitSheet.comp.test.tsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ describe('ExchangeApprovalLimitSheet', () => {
6262
expect(getByText('Unlimited')).toBeTruthy();
6363
expect(
6464
getByText(
65-
"Skip future approval requests and save on fees by approving unlimited USDC. Mercuryo will have full access to your USDC, and your funds may be at risk if they're ever compromised.",
65+
'Approve unlimited USDC to skip future approval requests and reduce fees. Only use this option if you trust Mercuryo, as it will have access to all your USDC.',
6666
),
6767
).toBeTruthy();
6868
});
@@ -73,7 +73,7 @@ describe('ExchangeApprovalLimitSheet', () => {
7373
expect(getByText('200.32 USDC')).toBeTruthy();
7474
expect(
7575
getByText(
76-
"Approve only the amount needed for this swap. You'll need to approve again (and pay a fee) for future swaps, but this reduces risk by keeping you in full control of your USDC.",
76+
"Approve only the amount needed for this swap. This helps reduce risk, but you'll need to approve again (and pay a fee) for future swaps.",
7777
),
7878
).toBeTruthy();
7979
});
@@ -90,7 +90,7 @@ describe('ExchangeApprovalLimitSheet', () => {
9090

9191
expect(
9292
getByText(
93-
"Skip future approval requests and save on fees by approving unlimited USDC. Mercuryo will have full access to your USDC, and your funds may be at risk if they're ever compromised.",
93+
'Approve unlimited USDC to skip future approval requests and reduce fees. Only use this option if you trust Mercuryo, as it will have access to all your USDC.',
9494
),
9595
).toBeTruthy();
9696
});

‎suite-native/module-trading/src/components/exchange/receive/__tests__/ExchangeReceiveCard.comp.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ describe('ExchangeReceiveCard', () => {
3434
const { getByText, getByLabelText } = await renderExchangeBuyCard();
3535

3636
expect(getByText('You get')).toBeOnTheScreen();
37-
expect(getByLabelText('Select coin')).toHaveTextContent(/USDC/);
37+
expect(getByLabelText('Select asset')).toHaveTextContent(/USDC/);
3838
expect(getByLabelText('Network name')).toHaveTextContent('Ethereum');
3939
expect(getByLabelText('You get')).toHaveDisplayValue('0.00083554');
4040
expect(getByText('Balance:')).toBeOnTheScreen();

‎suite-native/module-trading/src/components/exchange/receive/__tests__/ExchangeTradeableAssetPicker.comp.test.tsx‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,10 @@ describe('ExchangeTradeableAssetPicker', () => {
4242
form = await renderFormHook();
4343
});
4444

45-
it('should render "Select coin" button with caret', async () => {
45+
it('should render "Select asset" button with caret', async () => {
4646
const { getByLabelText } = await renderTradeableAssetPicker();
4747

48-
expect(getByLabelText('Select coin')).toHaveTextContent(/^Select coin.$/);
48+
expect(getByLabelText('Select asset')).toHaveTextContent(/^Select asset.$/);
4949
});
5050

5151
it('should render bottom sheet with all assets', async () => {

‎suite-native/module-trading/src/components/exchange/send/__tests__/ExchangeSendCard.comp.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ describe('ExchangeSendCard', () => {
3939

4040
expect(getByText('You pay')).toBeOnTheScreen();
4141
expect(getByText('$99.00')).toBeOnTheScreen();
42-
expect(getByLabelText('Select coin')).toHaveTextContent(/USDC/);
42+
expect(getByLabelText('Select asset')).toHaveTextContent(/USDC/);
4343
expect(getByLabelText('Network name')).toHaveTextContent('Ethereum');
4444
expect(getByLabelText('You pay')).toHaveDisplayValue('100');
4545
expect(getByText('Balance:')).toBeOnTheScreen();

‎suite-native/module-trading/src/components/fees/__tests__/TradingFeesForm.comp.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ describe('TradingFeesForm', () => {
8080

8181
expect(getByText('Transaction fee')).toBeTruthy();
8282
expect(
83-
getByText('Fees are paid directly to network miners for processing your transactions.'),
83+
getByText('Fees are paid directly to validators for processing your transactions.'),
8484
).toBeTruthy();
8585
});
8686

‎suite-native/module-trading/src/components/general/__tests__/SelectTradeableAssetButton.comp.test.tsx‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,28 +4,28 @@ import { adaAsset } from '../../../__fixtures__/tradeableAssets';
44
import { SelectTradeableAssetButton } from '../SelectTradeableAssetButton';
55

66
describe('SelectTradeableAssetButton', () => {
7-
it('should render "Select coin" when no network is selected', () => {
7+
it('should render "Select asset" when no network is selected', () => {
88
const { getByLabelText } = renderWithBasicProvider(
99
<SelectTradeableAssetButton onPress={jest.fn()} selectedAsset={undefined} caret />,
1010
);
1111

12-
const button = getByLabelText('Select coin');
13-
expect(button).toHaveTextContent(/^Select coin.$/);
12+
const button = getByLabelText('Select asset');
13+
expect(button).toHaveTextContent(/^Select asset.$/);
1414
});
1515

1616
it('should render TradeableAssetButton when network is selected', () => {
1717
const { getByLabelText } = renderWithBasicProvider(
1818
<SelectTradeableAssetButton onPress={jest.fn()} selectedAsset={adaAsset} caret />,
1919
);
20-
const button = getByLabelText('Select coin');
20+
const button = getByLabelText('Select asset');
2121
expect(button).toHaveTextContent(/^ADA.$/);
2222
});
2323

2424
it('should not display caret when caret prop is falsy', () => {
2525
const { getByLabelText } = renderWithBasicProvider(
2626
<SelectTradeableAssetButton onPress={jest.fn()} selectedAsset={adaAsset} />,
2727
);
28-
const button = getByLabelText('Select coin');
28+
const button = getByLabelText('Select asset');
2929
expect(button).toHaveTextContent('ADA');
3030
});
3131
});

‎suite-native/module-trading/src/components/sell/__tests__/SellCard.comp.test.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('SellCard', () => {
4545

4646
expect(getByText('You pay')).toBeOnTheScreen();
4747
expect(getByText('$99.00')).toBeOnTheScreen();
48-
expect(getByLabelText('Select coin')).toHaveTextContent(/USDC/);
48+
expect(getByLabelText('Select asset')).toHaveTextContent(/USDC/);
4949
expect(getByLabelText('Network name')).toHaveTextContent('Ethereum');
5050
expect(getByLabelText('You pay')).toHaveDisplayValue('100');
5151
expect(getByText('Balance:')).toBeOnTheScreen();

0 commit comments

Comments
 (0)