Skip to content

Show account verification warning on merge request details page

Daniel Tian requested to merge show-account-verification-warning-for-merge-request-details into master

What does this MR do and why?

For unconfirmed accounts (accounts that did not perform either e-mail, phone, or credit card verification), we are adding restrictions so that only one pipeline from that account can be run at the same time. This MR adds a warning alert to the MR details page that informs the user of this restriction, when they are hit by this restriction.
ksnip_20240423-135852

How to set up and validate locally

The backend is not implemented yet, so the variable that controls whether the alert is shown is currently stubbed out. Apply this patch, then go to any MR to see the alert:

Index: app/views/projects/merge_requests/_page.html.haml
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/app/views/projects/merge_requests/_page.html.haml b/app/views/projects/merge_requests/_page.html.haml
--- a/app/views/projects/merge_requests/_page.html.haml	(revision 13c99e9e6370131c3e6ac456a8cc482424c91f97)
+++ b/app/views/projects/merge_requests/_page.html.haml	(date 1713916618551)
@@ -12,7 +12,7 @@
 - add_page_specific_style 'page_bundles/pipelines'
 - add_page_specific_style 'page_bundles/reports'
 - add_page_specific_style 'page_bundles/ci_status'
-- identity_verification_required = false
+- identity_verification_required = true
 
 - add_page_startup_api_call @endpoint_metadata_url
 - add_page_startup_api_call @pinned_file_url if @pinned_file_url

Related to https://gitlab.com/gitlab-org/modelops/anti-abuse/team-tasks/-/issues/651

Edited by Daniel Tian

Merge request reports