From 84ff7e61e053d86a9e2d90729b15c536c3a03518 Mon Sep 17 00:00:00 2001
From: Rodrigo Arze <ral77ro@gmail.com>
Date: Tue, 25 Feb 2025 23:17:56 -0400
Subject: [PATCH 1/4] Memo main component, improve initial loader bg and bump
 version

---
 kits/connect/package.json        |   2 +-
 kits/connect/src/embed-react.tsx | 142 +++++++++++++++++--------------
 2 files changed, 78 insertions(+), 66 deletions(-)

diff --git a/kits/connect/package.json b/kits/connect/package.json
index 756043b07..e3b9381d5 100644
--- a/kits/connect/package.json
+++ b/kits/connect/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@openint/connect",
-  "version": "0.2.19",
+  "version": "0.2.22",
   "sideEffects": false,
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
diff --git a/kits/connect/src/embed-react.tsx b/kits/connect/src/embed-react.tsx
index 956551149..c1c510279 100644
--- a/kits/connect/src/embed-react.tsx
+++ b/kits/connect/src/embed-react.tsx
@@ -10,72 +10,82 @@ export interface OpenIntConnectEmbedProps
 
 const DEFAULT_HEIGHT = 500
 const DEFAULT_WIDTH = 350
-export const OpenIntConnectEmbed = React.forwardRef(
-  (
-    {baseUrl, params, onReady, ...iframeProps}: OpenIntConnectEmbedProps,
-    forwardedRef: React.ForwardedRef<HTMLIFrameElement>,
-  ) => {
-    const url = getIFrameUrl({baseUrl, params})
-    const [loading, setLoading] = React.useState(true)
+export const OpenIntConnectEmbed = React.memo(
+  React.forwardRef(
+    (
+      props: OpenIntConnectEmbedProps,
+      forwardedRef: React.ForwardedRef<HTMLIFrameElement>,
+    ) => {
+      const {baseUrl, params, onReady, ...iframeProps} = props
 
-    React.useEffect(() => {
-      if (
-        typeof iframeProps.height === 'number' &&
-        iframeProps.height < DEFAULT_HEIGHT
-      ) {
-        console.warn(
-          'Optimal height for Connect is 500px. Using 500px instead.',
-        )
-      }
-      if (typeof iframeProps.width === 'number' && iframeProps.width < 350) {
-        console.warn('Minimum width for Connect is 350px. Using 350px instead.')
-      }
-    }, [iframeProps.height, iframeProps.width])
+      const url = React.useMemo(
+        () => getIFrameUrl({baseUrl, params}),
+        [baseUrl, params],
+      )
 
-    const height =
-      typeof iframeProps.height === 'number'
-        ? iframeProps.height > DEFAULT_HEIGHT
-          ? iframeProps.height
-          : DEFAULT_HEIGHT
-        : iframeProps.height
+      const [loading, setLoading] = React.useState(true)
 
-    const width =
-      typeof iframeProps.width === 'number'
-        ? iframeProps.width > DEFAULT_WIDTH
-          ? iframeProps.width
-          : DEFAULT_WIDTH
-        : iframeProps.width || DEFAULT_WIDTH
+      React.useEffect(() => {
+        if (
+          typeof iframeProps.height === 'number' &&
+          iframeProps.height < DEFAULT_HEIGHT
+        ) {
+          console.warn(
+            'Optimal height for Connect is 500px. Using 500px instead.',
+          )
+        }
+        if (typeof iframeProps.width === 'number' && iframeProps.width < 350) {
+          console.warn(
+            'Minimum width for Connect is 350px. Using 350px instead.',
+          )
+        }
+      }, [iframeProps.height, iframeProps.width])
 
-    // Add a more reliable way to know iframe has fully finished loading
-    // by sending message from iframe to parent when ready
-    return (
-      <div className="connect-embed-wrapper">
-        <div className={`spinner-container ${loading ? 'loading' : 'loaded'}`}>
-          <svg className="spinner" viewBox="0 0 50 50">
-            <circle
-              className="path"
-              cx="25"
-              cy="25"
-              r="20"
-              fill="none"
-              strokeWidth="5"></circle>
-          </svg>
-        </div>
-        <iframe
-          name="openint-connect-frame"
-          id="openint-connect-frame"
-          {...iframeProps}
-          ref={forwardedRef}
-          onLoad={() => {
-            setLoading(false)
-            onReady?.()
-          }}
-          src={url}
-          width={width}
-          height={height}
-        />
+      const height =
+        typeof iframeProps.height === 'number'
+          ? iframeProps.height > DEFAULT_HEIGHT
+            ? iframeProps.height
+            : DEFAULT_HEIGHT
+          : iframeProps.height
+
+      const width =
+        typeof iframeProps.width === 'number'
+          ? iframeProps.width > DEFAULT_WIDTH
+            ? iframeProps.width
+            : DEFAULT_WIDTH
+          : iframeProps.width || DEFAULT_WIDTH
 
-        <style>{`
+      // Add a more reliable way to know iframe has fully finished loading
+      // by sending message from iframe to parent when ready
+      return (
+        <div className="connect-embed-wrapper">
+          <div
+            className={`spinner-container ${loading ? 'loading' : 'loaded'}`}>
+            <svg className="spinner" viewBox="0 0 50 50">
+              <circle
+                className="path"
+                cx="25"
+                cy="25"
+                r="20"
+                fill="none"
+                strokeWidth="5"></circle>
+            </svg>
+          </div>
+          <iframe
+            name="openint-connect-frame"
+            id="openint-connect-frame"
+            {...iframeProps}
+            ref={forwardedRef}
+            onLoad={() => {
+              setLoading(false)
+              onReady?.()
+            }}
+            src={url}
+            width={width}
+            height={height}
+          />
+
+          <style>{`
         .connect-embed-wrapper {
           display: flex;
           height: ${height}px;
@@ -91,7 +101,7 @@ export const OpenIntConnectEmbed = React.forwardRef(
           justify-content: center;
           align-items: center;
           height: 100%;
-          background: white;
+          background: ${params?.theme === 'dark' ? '#1C1C1C' : 'white'};
           transition: opacity 0.3s ease;
           max-width: ${width}px;
           max-height: ${height}px;
@@ -130,8 +140,10 @@ export const OpenIntConnectEmbed = React.forwardRef(
           }
         }
       `}</style>
-      </div>
-    )
-  },
+        </div>
+      )
+    },
+  ),
 )
+
 OpenIntConnectEmbed.displayName = 'OpenIntConnectEmbed'

From aa84b3f95dd4e369dccf827f7811b8219ca1807e Mon Sep 17 00:00:00 2001
From: Rodrigo Arze <ral77ro@gmail.com>
Date: Wed, 26 Feb 2025 09:26:37 -0400
Subject: [PATCH 2/4] Add spacing for no connections yet message

---
 packages/engine-frontend/components/ConnectionsTabContent.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/engine-frontend/components/ConnectionsTabContent.tsx b/packages/engine-frontend/components/ConnectionsTabContent.tsx
index ef15f643b..983a96887 100644
--- a/packages/engine-frontend/components/ConnectionsTabContent.tsx
+++ b/packages/engine-frontend/components/ConnectionsTabContent.tsx
@@ -102,10 +102,10 @@ export function ConnectionsTabContent({
       {connectionCount === 0 ? (
         <div className="flex flex-col p-4 pt-0">
           <div>
-            <p className="font-semibold text-foreground">
+            <p className="pt-4 font-semibold text-foreground">
               You have no connections yet
             </p>
-            <p className="text-sm text-foreground">
+            <p className="pt-2 text-sm text-foreground">
               Add a connection to start integrating and streamlining your
               workflow!
             </p>

From f115b1da2d93b257f372a0c8934c576bffb1a6b8 Mon Sep 17 00:00:00 2001
From: Rodrigo Arze <ral77ro@gmail.com>
Date: Wed, 26 Feb 2025 11:40:38 -0400
Subject: [PATCH 3/4] Update verticals to better fit integrations and clean
 other

---
 connectors/connector-discord/def.ts  |  1 +
 connectors/connector-github/def.ts   |  2 +-
 connectors/connector-gong/def.ts     |  2 +-
 connectors/connector-intercom/def.ts |  2 +-
 connectors/connector-jira/def.ts     |  2 +-
 connectors/connector-kustomer/def.ts |  2 +-
 connectors/connector-linear/def.ts   |  2 +-
 kits/cdk/verticals.ts                | 43 +++++++++++++++-------------
 8 files changed, 30 insertions(+), 26 deletions(-)

diff --git a/connectors/connector-discord/def.ts b/connectors/connector-discord/def.ts
index 5d68765e7..f5921f987 100644
--- a/connectors/connector-discord/def.ts
+++ b/connectors/connector-discord/def.ts
@@ -17,6 +17,7 @@ export const discordDef = {
   metadata: {
     displayName: 'Discord',
     stage: 'beta',
+    verticals: ['messaging'],
     logoUrl: '/_assets/logo-discord.svg',
     nangoProvider: 'discord',
   },
diff --git a/connectors/connector-github/def.ts b/connectors/connector-github/def.ts
index 2429211c1..339b9b35b 100644
--- a/connectors/connector-github/def.ts
+++ b/connectors/connector-github/def.ts
@@ -24,7 +24,7 @@ export const githubDef = {
   metadata: {
     displayName: 'GitHub',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['developer-tools'],
     logoUrl: '/_assets/logo-github.svg',
     nangoProvider: 'github',
   },
diff --git a/connectors/connector-gong/def.ts b/connectors/connector-gong/def.ts
index 2efc3e987..efabf719b 100644
--- a/connectors/connector-gong/def.ts
+++ b/connectors/connector-gong/def.ts
@@ -24,7 +24,7 @@ export const gongDef = {
   metadata: {
     displayName: 'Gong',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['sales-enablement'],
     logoUrl: '/_assets/logo-gong.svg',
     nangoProvider: 'gong',
   },
diff --git a/connectors/connector-intercom/def.ts b/connectors/connector-intercom/def.ts
index e2d1f499e..d7cc3f823 100644
--- a/connectors/connector-intercom/def.ts
+++ b/connectors/connector-intercom/def.ts
@@ -24,7 +24,7 @@ export const intercomDef = {
   metadata: {
     displayName: 'Intercom',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['communication'],
     logoUrl: '/_assets/logo-intercom.svg',
     nangoProvider: 'intercom',
   },
diff --git a/connectors/connector-jira/def.ts b/connectors/connector-jira/def.ts
index 037e50f74..e7504e720 100644
--- a/connectors/connector-jira/def.ts
+++ b/connectors/connector-jira/def.ts
@@ -24,7 +24,7 @@ export const jiraDef = {
   metadata: {
     displayName: 'Jira',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['ticketing'],
     logoUrl: '/_assets/logo-jira.svg',
     nangoProvider: 'jira',
   },
diff --git a/connectors/connector-kustomer/def.ts b/connectors/connector-kustomer/def.ts
index cef5428ca..42a97ff8b 100644
--- a/connectors/connector-kustomer/def.ts
+++ b/connectors/connector-kustomer/def.ts
@@ -24,7 +24,7 @@ export const kustomerDef = {
   metadata: {
     displayName: 'Kustomer',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['communication'],
     logoUrl: '/_assets/logo-kustomer.svg',
     // TODO: Update opensdks NangoProvider type to include kustomer
     // @ts-expect-error
diff --git a/connectors/connector-linear/def.ts b/connectors/connector-linear/def.ts
index 0f35bcc05..085781611 100644
--- a/connectors/connector-linear/def.ts
+++ b/connectors/connector-linear/def.ts
@@ -24,7 +24,7 @@ export const linearDef = {
   metadata: {
     displayName: 'Linear',
     stage: 'beta',
-    verticals: ['other'],
+    verticals: ['ticketing'],
     logoUrl: '/_assets/logo-linear.svg',
     nangoProvider: 'linear',
   },
diff --git a/kits/cdk/verticals.ts b/kits/cdk/verticals.ts
index c0f939cc3..8fc36e5d8 100644
--- a/kits/cdk/verticals.ts
+++ b/kits/cdk/verticals.ts
@@ -12,26 +12,7 @@ interface VerticalInfo {
 // Also maybe should be distributed in a metadata file associated with each unified api
 // impl.
 const _VERTICAL_BY_KEY = {
-  banking: {},
   accounting: {},
-  crm: {
-    name: 'CRM',
-    objects: ['account', 'contact', 'opportunity', 'lead', 'user'],
-  },
-  'sales-engagement': {},
-  engagement: {}, // TODO: merge me
-  commerce: {},
-  'expense-management': {},
-  enrichment: {},
-  database: {},
-  'flat-files-and-spreadsheets': {},
-  'file-storage': {},
-  streaming: {},
-  'personal-finance': {},
-  other: {},
-  hris: {},
-  payroll: {},
-  calendar: {},
   ats: {
     name: 'ATS',
     description: `Our secure API identifies employees and compensation by
@@ -40,9 +21,31 @@ const _VERTICAL_BY_KEY = {
                 one-way with no impact on original data.`,
     objects: ['job', 'offer', 'candidate', 'opening'],
   },
+  banking: {},
+  calendar: {},
+  commerce: {},
+  communication: {},
+  crm: {
+    name: 'CRM',
+    objects: ['account', 'contact', 'opportunity', 'lead', 'user'],
+  },
+  database: {},
+  'developer-tools': {},
   email: {},
+  engagement: {}, // TODO: merge me
+  enrichment: {},
+  'expense-management': {},
+  'file-storage': {},
+  'flat-files-and-spreadsheets': {},
+  hris: {},
   messaging: {},
-  communication: {},
+  other: {},
+  payroll: {},
+  'personal-finance': {},
+  'sales-enablement': {},
+  'sales-engagement': {},
+  streaming: {},
+  ticketing: {},
   wiki: {},
 } satisfies Record<string, VerticalInfo>
 

From 611b9f7c302c337630a7b8c40d1770a37806c901 Mon Sep 17 00:00:00 2001
From: Rodrigo Arze <ral77ro@gmail.com>
Date: Wed, 26 Feb 2025 11:49:58 -0400
Subject: [PATCH 4/4] Update docs

---
 kits/sdk/openapi.json       | 62 ++++++++++++++++++++-----------------
 kits/sdk/openapi.types.d.ts | 62 ++++++++++++++++++++-----------------
 2 files changed, 68 insertions(+), 56 deletions(-)

diff --git a/kits/sdk/openapi.json b/kits/sdk/openapi.json
index 649b02ae1..8364c5c33 100644
--- a/kits/sdk/openapi.json
+++ b/kits/sdk/openapi.json
@@ -1510,27 +1510,30 @@
                         "items": {
                           "type": "string",
                           "enum": [
-                            "banking",
                             "accounting",
+                            "ats",
+                            "banking",
+                            "calendar",
+                            "commerce",
+                            "communication",
                             "crm",
-                            "sales-engagement",
+                            "database",
+                            "developer-tools",
+                            "email",
                             "engagement",
-                            "commerce",
-                            "expense-management",
                             "enrichment",
-                            "database",
-                            "flat-files-and-spreadsheets",
+                            "expense-management",
                             "file-storage",
-                            "streaming",
-                            "personal-finance",
-                            "other",
+                            "flat-files-and-spreadsheets",
                             "hris",
-                            "payroll",
-                            "calendar",
-                            "ats",
-                            "email",
                             "messaging",
-                            "communication",
+                            "other",
+                            "payroll",
+                            "personal-finance",
+                            "sales-enablement",
+                            "sales-engagement",
+                            "streaming",
+                            "ticketing",
                             "wiki"
                           ]
                         }
@@ -8554,27 +8557,30 @@
             "items": {
               "type": "string",
               "enum": [
-                "banking",
                 "accounting",
+                "ats",
+                "banking",
+                "calendar",
+                "commerce",
+                "communication",
                 "crm",
-                "sales-engagement",
+                "database",
+                "developer-tools",
+                "email",
                 "engagement",
-                "commerce",
-                "expense-management",
                 "enrichment",
-                "database",
-                "flat-files-and-spreadsheets",
+                "expense-management",
                 "file-storage",
-                "streaming",
-                "personal-finance",
-                "other",
+                "flat-files-and-spreadsheets",
                 "hris",
-                "payroll",
-                "calendar",
-                "ats",
-                "email",
                 "messaging",
-                "communication",
+                "other",
+                "payroll",
+                "personal-finance",
+                "sales-enablement",
+                "sales-engagement",
+                "streaming",
+                "ticketing",
                 "wiki"
               ]
             }
diff --git a/kits/sdk/openapi.types.d.ts b/kits/sdk/openapi.types.d.ts
index 4e397f8d8..72b562721 100644
--- a/kits/sdk/openapi.types.d.ts
+++ b/kits/sdk/openapi.types.d.ts
@@ -659,27 +659,30 @@ export interface components {
       login_url?: string | null
       verticals?:
         | (
-            | 'banking'
             | 'accounting'
+            | 'ats'
+            | 'banking'
+            | 'calendar'
+            | 'commerce'
+            | 'communication'
             | 'crm'
-            | 'sales-engagement'
+            | 'database'
+            | 'developer-tools'
+            | 'email'
             | 'engagement'
-            | 'commerce'
-            | 'expense-management'
             | 'enrichment'
-            | 'database'
-            | 'flat-files-and-spreadsheets'
+            | 'expense-management'
             | 'file-storage'
-            | 'streaming'
-            | 'personal-finance'
-            | 'other'
+            | 'flat-files-and-spreadsheets'
             | 'hris'
-            | 'payroll'
-            | 'calendar'
-            | 'ats'
-            | 'email'
             | 'messaging'
-            | 'communication'
+            | 'other'
+            | 'payroll'
+            | 'personal-finance'
+            | 'sales-enablement'
+            | 'sales-engagement'
+            | 'streaming'
+            | 'ticketing'
             | 'wiki'
           )[]
         | null
@@ -2325,27 +2328,30 @@ export interface operations {
             isSource: boolean
             isDestination: boolean
             verticals: (
-              | 'banking'
               | 'accounting'
+              | 'ats'
+              | 'banking'
+              | 'calendar'
+              | 'commerce'
+              | 'communication'
               | 'crm'
-              | 'sales-engagement'
+              | 'database'
+              | 'developer-tools'
+              | 'email'
               | 'engagement'
-              | 'commerce'
-              | 'expense-management'
               | 'enrichment'
-              | 'database'
-              | 'flat-files-and-spreadsheets'
+              | 'expense-management'
               | 'file-storage'
-              | 'streaming'
-              | 'personal-finance'
-              | 'other'
+              | 'flat-files-and-spreadsheets'
               | 'hris'
-              | 'payroll'
-              | 'calendar'
-              | 'ats'
-              | 'email'
               | 'messaging'
-              | 'communication'
+              | 'other'
+              | 'payroll'
+              | 'personal-finance'
+              | 'sales-enablement'
+              | 'sales-engagement'
+              | 'streaming'
+              | 'ticketing'
               | 'wiki'
             )[]
             integrations: string[]