From 15ce9643e2c5d0bfe5cc96c017203548850643cf Mon Sep 17 00:00:00 2001
From: Darcy Ye <darcyye@silverhand.io>
Date: Tue, 28 Nov 2023 17:52:25 +0800
Subject: [PATCH] feat(console,phrases): add inline notification if email
 domain not configured (#4986)

---
 .../Experience/MultiInput/index.module.scss      |  6 ++++++
 .../Experience/index.module.scss                 |  4 ++++
 .../EnterpriseSsoDetails/Experience/index.tsx    | 16 +++++++++++++++-
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  3 +++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 .../admin-console/enterprise-sso-details.ts      |  5 +++++
 18 files changed, 98 insertions(+), 1 deletion(-)

diff --git a/packages/console/src/pages/EnterpriseSsoDetails/Experience/MultiInput/index.module.scss b/packages/console/src/pages/EnterpriseSsoDetails/Experience/MultiInput/index.module.scss
index be34dbdcd..af6108618 100644
--- a/packages/console/src/pages/EnterpriseSsoDetails/Experience/MultiInput/index.module.scss
+++ b/packages/console/src/pages/EnterpriseSsoDetails/Experience/MultiInput/index.module.scss
@@ -71,3 +71,9 @@
     outline-color: var(--color-focused-variant);
   }
 }
+
+.error {
+  border-color: var(--color-error);
+  font: var(--font-body-2);
+  margin-top: _.unit(1);
+}
diff --git a/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.module.scss b/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.module.scss
index 515346fa4..cdb2f36d4 100644
--- a/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.module.scss
+++ b/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.module.scss
@@ -11,3 +11,7 @@
   color: var(--color-error);
   margin-top: _.unit(1);
 }
+
+.inlineNotification {
+  margin-top: _.unit(6);
+}
diff --git a/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.tsx b/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.tsx
index ad3c8fff9..6fe170e06 100644
--- a/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.tsx
+++ b/packages/console/src/pages/EnterpriseSsoDetails/Experience/index.tsx
@@ -17,6 +17,7 @@ import DetailsForm from '@/components/DetailsForm';
 import FormCard from '@/components/FormCard';
 import UnsavedChangesAlertModal from '@/components/UnsavedChangesAlertModal';
 import FormField from '@/ds-components/FormField';
+import InlineNotification from '@/ds-components/InlineNotification';
 import Select from '@/ds-components/Select';
 import TextInput from '@/ds-components/TextInput';
 import useApi from '@/hooks/use-api';
@@ -98,7 +99,7 @@ function Experience({ data, isDeleted, onUpdated }: Props) {
     clearErrors,
     handleSubmit,
     register,
-    formState: { isDirty, isSubmitting, errors },
+    formState: { defaultValues, isDirty, isSubmitting, errors },
     reset,
   } = formMethods;
 
@@ -172,6 +173,11 @@ function Experience({ data, isDeleted, onUpdated }: Props) {
               error={errors.connectorName?.message}
             />
           </FormField>
+          {!defaultValues?.domains?.length && (
+            <InlineNotification className={styles.inlineNotification} severity="alert">
+              {t('enterprise_sso_details.configure_domain_field_info_text')}
+            </InlineNotification>
+          )}
           <FormField title="enterprise_sso_details.email_domain_field_name">
             <div className={styles.description}>
               {t('enterprise_sso_details.email_domain_field_description')}
@@ -179,6 +185,14 @@ function Experience({ data, isDeleted, onUpdated }: Props) {
             <Controller
               name="domains"
               control={control}
+              rules={{
+                validate: (value) => {
+                  if (value.length === 0) {
+                    return t('enterprise_sso_details.email_domain_field_required');
+                  }
+                  return true;
+                },
+              }}
               render={({ field: { onChange, value } }) => (
                 <MultiInput
                   values={value}
diff --git a/packages/phrases/src/locales/de/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/de/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/de/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/de/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/en/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/en/translation/admin-console/enterprise-sso-details.ts
index 8281fa9a7..ba527d113 100644
--- a/packages/phrases/src/locales/en/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/en/translation/admin-console/enterprise-sso-details.ts
@@ -37,6 +37,9 @@ const enterprise_sso_details = {
   upload_idp_metadata_title: 'Upload IdP metadata',
   upload_idp_metadata_description: 'Configure the metadata copied from the identity provider.',
   upload_idp_metadata_button_text: 'Upload metadata XML file',
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   upload_saml_idp_metadata_info_text_xml:
diff --git a/packages/phrases/src/locales/es/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/es/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/es/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/es/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/fr/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/fr/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/fr/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/fr/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/it/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/it/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/it/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/it/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/ja/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/ja/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/ja/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/ja/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/ko/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/ko/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/ko/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/ko/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/pl-pl/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/pl-pl/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/pl-pl/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/pl-pl/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/pt-br/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/pt-br/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/pt-br/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/pt-br/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/pt-pt/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/pt-pt/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/pt-pt/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/pt-pt/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/ru/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/ru/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/ru/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/ru/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/tr-tr/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/tr-tr/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/tr-tr/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/tr-tr/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/zh-cn/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/zh-cn/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/zh-cn/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/zh-cn/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/zh-hk/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/zh-hk/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/zh-hk/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/zh-hk/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */
diff --git a/packages/phrases/src/locales/zh-tw/translation/admin-console/enterprise-sso-details.ts b/packages/phrases/src/locales/zh-tw/translation/admin-console/enterprise-sso-details.ts
index 6d06a9e2d..612b9f65e 100644
--- a/packages/phrases/src/locales/zh-tw/translation/admin-console/enterprise-sso-details.ts
+++ b/packages/phrases/src/locales/zh-tw/translation/admin-console/enterprise-sso-details.ts
@@ -71,6 +71,11 @@ const enterprise_sso_details = {
   /** UNTRANSLATED */
   upload_idp_metadata_button_text: 'Upload metadata XML file',
   /** UNTRANSLATED */
+  configure_domain_field_info_text:
+    'Add email domain to guide enterprise users to their identity provider for Single Sign-on.',
+  /** UNTRANSLATED */
+  email_domain_field_required: 'Email domain is required to enable enterprise SSO.',
+  /** UNTRANSLATED */
   upload_saml_idp_metadata_info_text_url:
     'Paste the metadata URL from the identity provider to connect.',
   /** UNTRANSLATED */