Update SAML response alert

What does this MR do and why?

Update SAML response alert

  • use pajamas alert styling

Screenshots or screen recordings

I had trouble testing this alert because the feature set up is quite involved, but the code is not that complex so I would expect this to work. Also, I think it has been removed from the sidebar, so I am not even sure this feature is used anymore.

Before After
image Screen_Shot_2021-11-03_at_15.32.27
image image

Before Code:

diff --git a/app/views/dashboard/_projects_head.html.haml b/app/views/dashboard/_projects_head.html.haml
index fdaf2107686..41b2b556dbd 100644
--- a/app/views/dashboard/_projects_head.html.haml
+++ b/app/views/dashboard/_projects_head.html.haml
@@ -4,6 +4,8 @@
 = content_for :flash_message do
   = render 'shared/project_limit'
 
+.alert.alert-success
+  = s_('GroupSAML|Valid SAML Response')
 .page-title-holder.d-flex.align-items-center
   %h1.page-title= _('Projects')

After Code:

diff --git a/app/views/dashboard/_projects_head.html.haml b/app/views/dashboard/_projects_head.html.haml
index fdaf2107686..ab49e672123 100644
--- a/app/views/dashboard/_projects_head.html.haml
+++ b/app/views/dashboard/_projects_head.html.haml
@@ -4,6 +4,12 @@
 = content_for :flash_message do
   = render 'shared/project_limit'
 
+.gl-alert.gl-alert-success
+  .gl-alert-container
+    = sprite_icon('check-circle', css_class: 'gl-icon gl-alert-icon gl-alert-icon-no-title')
+    .gl-alert-content
+      .gl-alert-body
+        = s_('GroupSAML|Valid SAML Response')
 .page-title-holder.d-flex.align-items-center
   %h1.page-title= _('Projects')

How to set up and validate locally

Handbook docs

GDK SAML docs

MR introducing this feature

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

Edited by Alexander Turinske

Merge request reports

Loading