Skip to content

Add description diffs feature to the Registration Features program

Piotr Skorupa requested to merge psk-add-description-diffs-to-registration-features into master

What does this MR do and why?

Related to https://gitlab.com/gitlab-org/gitlab/-/issues/404703.

This adds viewing description change history (description_diffs in code) to the Registration Features program. This means that it's enabled as an incentive for self-managed instances that don't have a Premium license, if Service Ping is active.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

  1. Testing previous behavior:
    • Make sure you're on a Premium/Ultimate plan GitLab GDK instance
    • Try to view description change history for an Issue/MR/Epic (it should be enabled).
  2. When registration features are enabled:
    • Make sure you're on a free plan GitLab GDK instance (for example, remove current license or stub #current to return nil)
    • Make sure you have the registration features checkbox enabled (Admin -> Settings -> Metrics and profiling -> Usage statistics -> Enable Registration Features)
    • Try to view description change history for an Issue/MR/Epic (it should be enabled and possible to view).
  3. When registration features are disabled:
    • Make sure you're on a free plan GitLab GDK instance
    • Make sure you have the registration features checkbox disabled (Admin -> Settings -> Metrics and profiling -> Usage statistics -> Enable Registration Features)
    • Try to view description change history for an Issue/MR/Epic (it should be disabled).
  4. Make sure that the new text appears on the docs page:
    • run gdk restart gitlab-docs
    • go to _local_gitlab_docs_host_/ee/user/admin_area/settings/usage_statistics.html#registration-features-program and make sure the new section is on the page and the link works

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Piotr Skorupa

Merge request reports