diff --git a/wherehows-web/app/components/dataset-authors.ts b/wherehows-web/app/components/dataset-authors.ts index 77ac051316..46343c54eb 100644 --- a/wherehows-web/app/components/dataset-authors.ts +++ b/wherehows-web/app/components/dataset-authors.ts @@ -200,6 +200,8 @@ export default class DatasetAuthors extends Component { const owners = get(this, 'owners') || []; const { notify } = get(this, 'notifications'); + debugger; + if (ownerAlreadyExists(owners, { userName: newOwner.userName, source: newOwner.source })) { return void notify(NotificationEvent.info, { content: 'Owner has already been added to "confirmed" list' }); } diff --git a/wherehows-web/app/constants/notifications.ts b/wherehows-web/app/constants/notifications.ts index 70d5480a0f..5ac6b643ed 100644 --- a/wherehows-web/app/constants/notifications.ts +++ b/wherehows-web/app/constants/notifications.ts @@ -7,5 +7,5 @@ export const bannerAnimationSpeed = 0.6; /** * Base message for 2 factor auth banner message for the login screen. */ -export const showTwoFABannerMessage = +export const twoFABannerMessage = 'We have enabled 2-factor authentication. Login now requires your password + Symantec VIP Token.'; diff --git a/wherehows-web/app/controllers/login.ts b/wherehows-web/app/controllers/login.ts index 4002ad4202..394229a240 100644 --- a/wherehows-web/app/controllers/login.ts +++ b/wherehows-web/app/controllers/login.ts @@ -3,7 +3,7 @@ import { computed, get, setProperties, getProperties } from '@ember/object'; import ComputedProperty from '@ember/object/computed'; import { inject as service } from '@ember/service'; import Session from 'ember-simple-auth/services/session'; -import { showTwoFABannerMessage } from 'wherehows-web/constants/notifications'; +import { twoFABannerMessage } from 'wherehows-web/constants/notifications'; import BannerService from 'wherehows-web/services/banners'; import { NotificationEvent } from 'wherehows-web/services/notifications'; @@ -52,7 +52,7 @@ export default class Login extends Controller { // Once user has chosen to authenticate, then we remove the login banner (if it exists) since it will // no longer be relevant - banners.removeBanner(showTwoFABannerMessage, NotificationEvent['info']); + banners.removeBanner(twoFABannerMessage, NotificationEvent['info']); get(this, 'session') .authenticate('authenticator:custom-ldap', username, password) diff --git a/wherehows-web/app/routes/login.js b/wherehows-web/app/routes/login.js index deb9e3c010..f94329bc51 100644 --- a/wherehows-web/app/routes/login.js +++ b/wherehows-web/app/routes/login.js @@ -2,7 +2,7 @@ import Route from '@ember/routing/route'; import { get } from '@ember/object'; import { run } from '@ember/runloop'; import { inject as service } from '@ember/service'; -import { showTwoFABannerMessage } from 'wherehows-web/constants/notifications'; +import { twoFABannerMessage } from 'wherehows-web/constants/notifications'; export default Route.extend({ session: service(), @@ -59,6 +59,6 @@ export default Route.extend({ */ showTwoFABanner() { const banners = get(this, 'banners'); - banners.addBanner(showTwoFABannerMessage, 'info'); + banners.addBanner(twoFABannerMessage, 'info'); } });