Skip to content

Update saml response alert

Alexander Turinske requested to merge 354596-migrate-saml-alert into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Update saml response alert

  • use shared component

Changelog: other

EE: true

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

Before After
image Screen_Shot_2022-03-18_at_14.02.25

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Apply this patch
diff --git a/app/views/dashboard/_projects_head.html.haml b/app/views/dashboard/_projects_head.html.haml
index b94b14bf6bd..83139f390a9 100644
--- a/app/views/dashboard/_projects_head.html.haml
+++ b/app/views/dashboard/_projects_head.html.haml
@@ -4,6 +4,10 @@
 = content_for :flash_message do
   = render 'shared/project_limit'
 
+= render 'shared/global_alert',
+  variant: :success do
+  .gl-alert-body
+    = s_('GroupSAML|Valid SAML Response')
 .page-title-holder.d-flex.align-items-center
   %h1.page-title= _('Projects')
  1. Navigate to projects page

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 #354596 (closed)

Edited by Alexander Turinske

Merge request reports