Skip to content

Move Slack Events code to EE

Luke Duncalferequested to merge
371675-move-slack-events-api-to-ee into master

What does this MR do and why?

This MR moves the Slack Events API code to EE.

The Slack Events API was added in !86323 (merged) to FOSS.

The Slack Events API only works with the GitLab Slack App, which is currently SaaS-only.

The GitLab Slack App is used with the GitLab Slack Application integration, and the models for these (Integrations::GitlabSlackApplication and SlackInstallation) are only in EE.

The reason the models are in EE is because of the current limitation of the GitLab Slack App only working for SaaS.

In the near future, we plan to make it available to all self-managed customers, at which point the code will be moved to FOSS.

In the meantime, while building on the Slack Events API for #363635 (closed), it was discovered that due to some of the models being EE-only, it's impossible to continue to write passing tests for the feature while the API is in FOSS.

Issue #371675 (closed)

MR acceptance checklist

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

Related to #371675 (closed)

Edited by Luke Duncalfe

Merge request reports