Skip to content

Use public email for seat usage group members

Vijay Hawoldar requested to merge vij-billable-members-entity into master

What does this MR do?

As discussed in #292086 (closed) and the related issues, the logic behind whether or not to display a group member's email address in the seat usage page needs to change.

Previously it would use Group Managed Accounts to determine if it should or not, but now we are going to fall back to a more basic approach of using the public email address.

This MR addresses that by exposing the user's public_email (if present) - if it's not present, we display the private email message and new tooltip (see below).

The frontend Vue component checks if the email value is present or not to determine if it should be displayed, so no logic changes need to take place there (source).

Testing

In order to test this change, you'll likely need to do the following:

  • Enable the feature flag: Feature.enable(:api_billable_member_list)
  • Navigate to the seat usage view: /groups/your-group/-/seat_usage

Screenshots

Before After
Screenshot_2020-12-21_at_08.39.14 Screenshot_2020-12-21_at_08.41.02

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Vijay Hawoldar

Merge request reports