Skip to content

Move old integrations to Settings > General and introduce instance-level integrations

What does this MR do?

Context

In !25752 (merged), we moved some of the old "integrations" from Admin > Settings > Integrations to Admin > Settings > General. This was to make space for our new instance-level integrations (similar to project integrations - or as they were previously called - project services).

Changes

  • Remove feature flag instance_level_integrations.
  • Fix redirect path after saving of old integration forms.
  • Remove stubs of instance_level_integrations set to false in specs and make sure all of them pass.

Screenshots

Section Before After
Admin > Settings > General General_before General_after
Admin > Settings > Integrations Integrations_before Integrations_after

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 Justin Ho Tuan Duong

Merge request reports