diff options
author | Whymarrh Whitby <whymarrh.whitby@gmail.com> | 2018-07-28 06:35:26 +0800 |
---|---|---|
committer | Whymarrh Whitby <whymarrh.whitby@gmail.com> | 2018-07-28 22:29:51 +0800 |
commit | 0fd8862c5e942778763e170a36aeb7604472c5cd (patch) | |
tree | d30917ad3c38e4711f00d1a37d6b5800cc7f7850 | |
parent | 07c0cd59d4b670eafc5d6c20ab9f54faa841ee1c (diff) | |
download | tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar.gz tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar.bz2 tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar.lz tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar.xz tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.tar.zst tangerine-wallet-browser-0fd8862c5e942778763e170a36aeb7604472c5cd.zip |
Don't show an additional beta UI notification
When a user has opted-in to the new UI from the announcement, we don't
need an additional notification letting the user know that they've
switched.
-rw-r--r-- | old-ui/app/components/app-bar.js | 2 | ||||
-rw-r--r-- | old-ui/app/new-ui-annoucement.js | 2 | ||||
-rw-r--r-- | test/e2e/beta/from-import-beta-ui.spec.js | 6 | ||||
-rw-r--r-- | test/e2e/beta/metamask-beta-ui.spec.js | 4 |
4 files changed, 2 insertions, 12 deletions
diff --git a/old-ui/app/components/app-bar.js b/old-ui/app/components/app-bar.js index e65f942ae..4253c5f28 100644 --- a/old-ui/app/components/app-bar.js +++ b/old-ui/app/components/app-bar.js @@ -82,7 +82,7 @@ module.exports = class AppBar extends Component { AppBar.renderSpace(), h('span.banner__link', { async onClick () { - await dispatch(actions.setFeatureFlag('betaUI', true, 'BETA_UI_NOTIFICATION_MODAL')) + await dispatch(actions.setFeatureFlag('betaUI', true)) }, }, [ 'Now', diff --git a/old-ui/app/new-ui-annoucement.js b/old-ui/app/new-ui-annoucement.js index 504016b61..a20dae2f8 100644 --- a/old-ui/app/new-ui-annoucement.js +++ b/old-ui/app/new-ui-annoucement.js @@ -15,11 +15,9 @@ module.exports = class NewUiAnnouncement extends PureComponent { switchToNewUi = async () => { const flag = 'betaUI' const enabled = true - const notificationType = 'BETA_UI_NOTIFICATION_MODAL' await this.props.dispatch(actions.setFeatureFlag( flag, enabled, - notificationType, )) await this.close() } diff --git a/test/e2e/beta/from-import-beta-ui.spec.js b/test/e2e/beta/from-import-beta-ui.spec.js index 92e5be715..06e669e5b 100644 --- a/test/e2e/beta/from-import-beta-ui.spec.js +++ b/test/e2e/beta/from-import-beta-ui.spec.js @@ -121,10 +121,6 @@ describe('Using MetaMask with an existing account', function () { await driver.switchTo().window(appTab) await delay(tinyDelayMs) - const okayBtn = await findElement(driver, By.xpath("//button[contains(text(), 'Ok')]")) - await okayBtn.click() - await delay(regularDelayMs) - await loadExtension(driver, extensionId) await delay(regularDelayMs) @@ -238,7 +234,7 @@ describe('Using MetaMask with an existing account', function () { await localhost.click() await delay(largeDelayMs * 2) }) - + it('choose Create Account from the account menu', async () => { await driver.findElement(By.css('.account-menu__icon')).click() await delay(regularDelayMs) diff --git a/test/e2e/beta/metamask-beta-ui.spec.js b/test/e2e/beta/metamask-beta-ui.spec.js index 6a169de76..98132e68c 100644 --- a/test/e2e/beta/metamask-beta-ui.spec.js +++ b/test/e2e/beta/metamask-beta-ui.spec.js @@ -120,10 +120,6 @@ describe('MetaMask', function () { await driver.switchTo().window(appTab) await delay(tinyDelayMs) - const okayBtn = await findElement(driver, By.xpath("//button[contains(text(), 'Ok')]")) - await okayBtn.click() - await delay(regularDelayMs) - await loadExtension(driver, extensionId) await delay(regularDelayMs) |