Skip to content

Merge train checkbox text update

Marcel Amirault requested to merge merge-train-checkbox into master

What does this MR do?

As an MVC for #212875 (closed), this updates the checkbox text based on the suggested style. I modified the subtext slightly to try to incorporate some of the suggestions that were brought up in the discussions.

I also had to remove the links from the suggestion, for this MVC, as links in checkbox text is complicated to get working with the specs. I think there is still a lot of benefit to to just improving the text to start. If we need to add more links, we can iterate.

Closes #212875 (closed)

Screenshots

Before

Screen_Shot_2020-06-08_at_22.49.26

After

Screen_Shot_2020-06-09_at_10.16.23

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
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading