Skip to content

Commit

Permalink
Rename 'History' tab to 'Activity' (#8785)
Browse files Browse the repository at this point in the history
'Activity' is a better name for this tab because it contains more than
just transactions. Signature requests are also included, and more non-
transaction activity may be included in the future.
  • Loading branch information
Gudahtt authored Jun 11, 2020
1 parent 058c63c commit 4ac4790
Show file tree
Hide file tree
Showing 8 changed files with 11 additions and 8 deletions.
3 changes: 3 additions & 0 deletions app/_locales/en/messages.json
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,9 @@
"accountSelectionRequired": {
"message": "You need to select an account!"
},
"activity": {
"message": "Activity"
},
"activityLog": {
"message": "activity log"
},
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/address-book.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ describe('MetaMask', function () {
})

it('finds the transaction in the transactions list', async function () {
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.wait(async () => {
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
return confirmedTxes.length === 1
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/from-import-ui.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ describe('Using MetaMask with an existing account', function () {
})

it('finds the transaction in the transactions list', async function () {
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.wait(async () => {
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
return confirmedTxes.length === 1
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/metamask-responsive-ui.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ describe('MetaMask', function () {
})

it('finds the transaction in the transactions list', async function () {
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.wait(async () => {
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
return confirmedTxes.length === 1
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/metamask-ui.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ describe('MetaMask', function () {
})

it('finds the transaction in the transactions list', async function () {
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.wait(async () => {
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
return confirmedTxes.length === 1
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/send-edit.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ describe('Using MetaMask with an existing account', function () {
})

it('finds the transaction in the transactions list', async function () {
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.wait(async () => {
const confirmedTxes = await driver.findElements(By.css('.transaction-list__completed-transactions .transaction-list-item'))
return confirmedTxes.length === 1
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/tests/simple-send.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ describe('MetaMask Browser Extension', function () {
await amountField.sendKeys('1')
await driver.clickElement(By.css('[data-testid="page-container-footer-next"]'))
await driver.clickElement(By.css('[data-testid="page-container-footer-next"]'))
await driver.clickElement(By.css('[data-testid="home__history-tab"]'))
await driver.clickElement(By.css('[data-testid="home__activity-tab"]'))
await driver.findElement(By.css('.transaction-list-item'))
})
})
Expand Down
4 changes: 2 additions & 2 deletions ui/app/pages/home/home.component.js
Original file line number Diff line number Diff line change
Expand Up @@ -247,8 +247,8 @@ export default class Home extends PureComponent {
<Tab
activeClassName="home__tab--active"
className="home__tab"
data-testid="home__history-tab"
name={t('history')}
data-testid="home__activity-tab"
name={t('activity')}
>
<TransactionList />
</Tab>
Expand Down

0 comments on commit 4ac4790

Please sign in to comment.