Skip to content

Conversation

@maxceem
Copy link
Collaborator

@maxceem maxceem commented Jul 17, 2019

No description provided.

@maxceem maxceem requested a review from vikasrohit July 17, 2019 11:23
} */
const powerUserRoles = [ROLE_CONNECT_COPILOT, ROLE_CONNECT_MANAGER, ROLE_ADMINISTRATOR, ROLE_CONNECT_ADMIN]
const isCustomer = _.intersection(user.roles, powerUserRoles).length === 0
const role = isCustomer ? 'Customer' : 'Member'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@maxceem just a note, it was showing me as Customer even when I have power user roles.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vikasrohit I guess it was an issue with logic. If I'm not mistaken we didn't have direct requirements from the product team on what text should be shown here. So I've simplified logic to always show User since for now.

@vikasrohit vikasrohit merged commit c55598f into master Jul 17, 2019
@vikasrohit vikasrohit deleted the hotifx/post-release-2-4-13 branch September 18, 2019 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants