Warn user conditionally if they need to update their Slack application in order to use new notifications functionality
Context
This is part of &8670 (closed)
There are currently 2 Slack integrations:
- Slack notifications: set up starting point is in Slack app. Sends notifications from GitLab to Slack
- Slack application: set up starting point is GitLab. Allows users to type
/commands
to do actions in GitLab
Slack notifications will be deprecated and its functionality will be brought into Slack application.
As a first step, we will be adding notification functionality to Slack application (#372410 (closed))
Problem
In order for the notifications functionality in Slack application to work, the user needs to update to the latest version of Slack application. In #374720 (closed), we will be able to track which version of Slack application the user is on.
When we have that information, we will be able to let the user know if they need to update their Slack application in order to have notification functionality work.
Proposal
When the user needs to update Slack, tell them and do not allow them to use notification functionality until they do so:
Availability & Testing
Since the initial frontend implementation relies on backend work that hasn't been completed yet, suggestions are:
- Write frontend component or integration specs that validate that the alert shows/is hidden and the fields are disabled or not based on a mocked response from the backend. This will also help clarify the contract that the frontend will need.
- After the backend feature is implemented, consider following up with a feature spec that mocks the api responses from Slack. Please feel free to ping @sgregory2 for assistance needed on this.