Skip to content

Allow SlackService while migrating

username-removed-443319 requested to merge allow-slack-service into master
  1. Make the existing migration converting SlackService -> SlackNotificationService a no-op.
  2. Allow SlackService as an alternative class name for SlackNotificationService. This also means that #to_param returns the same thing for both classes, so the UI will work in either case.
  3. Create a post-deployment migration to update the existing service names in batches.

In a future release, we can remove the dummy SlackService class and migrate any other data from SlackService -> SlackNotificationService.

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25843.

Merge request reports