aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWhymarrh Whitby <whymarrh.whitby@gmail.com>2018-07-28 06:35:26 +0800
committerWhymarrh Whitby <whymarrh.whitby@gmail.com>2018-07-28 22:29:51 +0800
commit0fd8862c5e942778763e170a36aeb7604472c5cd (patch)
treed30917ad3c38e4711f00d1a37d6b5800cc7f7850
parent07c0cd59d4b670eafc5d6c20ab9f54faa841ee1c (diff)
downloadtangerine-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.js2
-rw-r--r--old-ui/app/new-ui-annoucement.js2
-rw-r--r--test/e2e/beta/from-import-beta-ui.spec.js6
-rw-r--r--test/e2e/beta/metamask-beta-ui.spec.js4
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)