From fbb271caeefb242c182bad2d51163adbedc174b1 Mon Sep 17 00:00:00 2001
From: RachelElysia <71795832+RachelElysia@users.noreply.github.com>
Date: Fri, 22 Mar 2024 14:52:54 -0400
Subject: [PATCH] Fleet UI: Calendar settings iterations (#17779)

---
 .../cards/Calendars/Calendars.tsx             | 33 +++++++++++++++++--
 .../cards/Calendars/_styles.scss              |  5 +++
 2 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/frontend/pages/admin/IntegrationsPage/cards/Calendars/Calendars.tsx b/frontend/pages/admin/IntegrationsPage/cards/Calendars/Calendars.tsx
index de7c79a139..da22ea4801 100644
--- a/frontend/pages/admin/IntegrationsPage/cards/Calendars/Calendars.tsx
+++ b/frontend/pages/admin/IntegrationsPage/cards/Calendars/Calendars.tsx
@@ -33,7 +33,7 @@ const API_KEY_JSON_PLACEHOLDER = `{
   "type": "service_account",
   "project_id": "fleet-in-your-calendar",
   "private_key_id": "<private key id>",
-  "private_key": "-----BEGIN PRIVATE KEY-----\n<private key>\n-----END PRIVATE KEY-----\n",
+  "private_key": "-----BEGIN PRIVATE KEY----\\n<private key>\\n-----END PRIVATE KEY-----\\n",
   "client_email": "fleet-calendar-events@fleet-in-your-calendar.iam.gserviceaccount.com",
   "client_id": "<client id>",
   "auth_uri": "https://accounts.google.com/o/oauth2/auth",
@@ -58,6 +58,16 @@ interface ICalendarsFormData {
   apiKeyJson?: string;
 }
 
+// Used to surface error.message in UI of unknown error type
+type ErrorWithMessage = {
+  message: string;
+  [key: string]: unknown;
+};
+
+const isErrorWithMessage = (error: unknown): error is ErrorWithMessage => {
+  return (error as ErrorWithMessage).message !== undefined;
+};
+
 const baseClass = "calendars-integration";
 
 const Calendars = (): JSX.Element => {
@@ -103,7 +113,18 @@ const Calendars = (): JSX.Element => {
       errors.apiKeyJson = "API key JSON must be present";
     }
     if (!curFormData.domain && !!curFormData.apiKeyJson) {
-      errors.apiKeyJson = "Domain must be present";
+      errors.domain = "Domain must be present";
+    }
+    if (curFormData.apiKeyJson) {
+      try {
+        JSON.parse(curFormData.apiKeyJson);
+      } catch (e: unknown) {
+        if (isErrorWithMessage(e)) {
+          errors.apiKeyJson = e.message.toString();
+        } else {
+          throw e;
+        }
+      }
     }
     return errors;
   };
@@ -277,6 +298,7 @@ const Calendars = (): JSX.Element => {
                     placeholder={API_KEY_JSON_PLACEHOLDER}
                     ignore1password
                     inputClassName={`${baseClass}__api-key-json`}
+                    error={formErrors.apiKeyJson}
                   />
                   <InputField
                     label="Primary domain"
@@ -301,12 +323,13 @@ const Calendars = (): JSX.Element => {
                         />
                       </>
                     }
+                    error={formErrors.domain}
                   />
                   <Button
                     type="submit"
                     variant="brand"
                     disabled={Object.keys(formErrors).length > 0}
-                    className="save-loading button-wrap"
+                    className="save-loading"
                     isLoading={isUpdatingSettings}
                   >
                     Save
@@ -378,6 +401,10 @@ const Calendars = (): JSX.Element => {
               </li>
             </ul>
           </p>
+          <p>
+            You&apos;re ready to automatically schedule calendar events for end
+            users.
+          </p>
         </div>
       </>
     );
diff --git a/frontend/pages/admin/IntegrationsPage/cards/Calendars/_styles.scss b/frontend/pages/admin/IntegrationsPage/cards/Calendars/_styles.scss
index 7045a443ef..01db771e00 100644
--- a/frontend/pages/admin/IntegrationsPage/cards/Calendars/_styles.scss
+++ b/frontend/pages/admin/IntegrationsPage/cards/Calendars/_styles.scss
@@ -4,6 +4,10 @@
     color: $core-fleet-black;
   }
 
+  p {
+    margin: $pad-large 0;
+  }
+
   ui {
     margin-block-start: $pad-small;
   }
@@ -30,6 +34,7 @@
 
   #oauth-scopes {
     font-family: "SourceCodePro", $monospace;
+    color: $core-fleet-black;
     min-height: 80px;
     padding: $pad-medium;
     padding-right: $pad-xxlarge;