Skip to content

Update new user invitation email with permission information

What does this MR do?

  • This creates an experiment, using gitlab-experiment as a multi-variant addition of permission_info where we show the avatar with group or project and the role description.
  • Introduces the concept of rollout strategies and implements round-robin strategy.

We'll roll out the experiment with a feature flag members_invite_email for round-robin selection of variants and control to see if it improves the acceptance rate to groups.

We'll achieve the even split between control, avatar and permission_info by enabling the feature flag of members_invite_email and then doing a round-robin between control, avatar and permission_info.

How to test?

  • In Rails console(bundle exec rails console), issue this command to put yourself in the experiment
Feature.enable(:members_invite_email)
  • send an email 3 times to cycle through the variants. May need to send a few invites before the cache clears and updates sidekiq to see the change on feature flag state changes.
  • To completely disable the experiment, disabled the feature flag.
Feature.disable(:members_invite_email)

Screenshots (strongly suggested)

source variant email
project control Screen_Shot_2021-02-05_at_12.24.48_PM
project permission_info Screen_Shot_2021-02-05_at_12.23.55_PM
project avatar Screen_Shot_2021-02-05_at_12.25.22_PM
group control Screen_Shot_2021-02-05_at_12.18.40_PM
group permission_info Screen_Shot_2021-02-05_at_12.21.26_PM
group avatar Screen_Shot_2021-02-05_at_12.22.47_PM

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #296967

Edited by Doug Stull

Merge request reports