From 0fd8862c5e942778763e170a36aeb7604472c5cd Mon Sep 17 00:00:00 2001 From: Whymarrh Whitby Date: Fri, 27 Jul 2018 20:05:26 -0230 Subject: 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. --- old-ui/app/components/app-bar.js | 2 +- old-ui/app/new-ui-annoucement.js | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) (limited to 'old-ui/app') 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() } -- cgit v1.2.3