Draft: Create create/update status checks UI
What does this MR do?
With #327638 (closed) we are pivoting the status checks from being developed as part of the MR approval rules section, to its own dedicated section. This is because status checks don't "approve" an MR but rather give the MR author/merger an idea as to whether the MR passes any external processes. Putting it in a separate section helps keep the distinction that you don't need a status check to pass to merge an MR like you do approval rules.
This MR creates the create/update status checks UI by:
- Passing the projects ID to the store
- Setting up modals for the add/edit buttons
- Loading a status checks form within the modals using a shared form between them
- Set up store actions to submit the status check via the REST API
Screenshots (strongly suggested)
View | Light mode | Dark mode | Errors |
---|---|---|---|
Create | |||
Update |
Setup & testing
Note: Right now, deleting status checks is functioning within the approval rules section as per previous work. Use this area to delete the status checks for now until future MR's move the functionality to the new area
- Enable the feature flag
:ff_compliance_approval_gates
- Go to
[HOST]/gitlab-org/gitlab-test/edit
and expand theMerge requests
section - Click the status checks add button and try breaking the values in the form
- Add a new status check
- Check that
Status checks
contains the added status check - Click the status checks edit button and try breaking the values in the form
- Update the status check
- Check that
Status checks
contains the updated status check
Does this MR meet the acceptance criteria?
Conformity
-
📋 Does this MR need a changelog?-
I have included a changelog entry. -
I have not included a changelog entry because this is behind the :ff_compliance_approval_gates
feature flag.
-
- [-] Documentation (if required)
-
Code review guidelines -
Merge request performance guidelines -
Style guides - [-] Database guides
-
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers - [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done
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
Related to #327638 (closed)