Skip to content

Experiment: Define invite email preview text

What does this MR do?

  • Adds an experiment that will utilize preview text functionality on email clients to show an improved preview experience before opening.
  • Adds go-to actions for gmail on Join now

Screenshots or Screencasts (strongly suggested)

Proof of the div being present for the preview text

Screen_Shot_2021-08-02_at_10.06.27_AM

Go-to action validation using https://www.google.com/webmasters/markup-tester/

With localhost URL that of course shows invalid since google looks on internet for the URL

Screen_Shot_2021-07-30_at_12.18.25_PM

With a GitLab URL to show gitlab.com is considered valid

Screen_Shot_2021-07-30_at_12.19.09_PM

How to setup and validate locally (strongly suggested)

  1. In rails console enable the experiment fully
    Feature.enable(:invite_email_preview_text)
  2. Visit the mail previewer: http://127.0.0.1:3000/rails/mailers/notify/member_invited_email
  3. Inspect dom for expected changes as seen in screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • 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 #333024

Edited by Doug Stull

Merge request reports