fixup: only track schema export when copied or the download button is clicked

Author: Anemy

packages/compass-schema/src/components/export-schema-modal.tsx

@@ -21,6 +21,7 @@ import {
   cancelExportSchema,
   changeExportSchemaFormat,
   closeExportSchema,
+  trackSchemaExported,
   type SchemaFormat,
   type ExportStatus,
 } from '../stores/schema-export-reducer';
@@ -75,6 +76,7 @@ const ExportSchemaModal: React.FunctionComponent<{
   onCancelSchemaExport: () => void;
   onChangeSchemaExportFormat: (format: SchemaFormat) => Promise<void>;
   onClose: () => void;
+  onExportedSchemaCopied: () => void;
 }> = ({
   errorMessage,
   exportStatus,
@@ -84,6 +86,7 @@ const ExportSchemaModal: React.FunctionComponent<{
   onCancelSchemaExport,
   onChangeSchemaExportFormat,
   onClose,
+  onExportedSchemaCopied,
 }) => {
   const onFormatOptionSelected = useCallback(
     (event: ChangeEvent<HTMLInputElement>) => {
@@ -143,6 +146,7 @@ const ExportSchemaModal: React.FunctionComponent<{
               language="json"
               className={codeStyles}
               copyable={true}
+              onCopy={onExportedSchemaCopied}
             >
               {exportedSchema ?? 'Empty'}
             </Code>
@@ -163,7 +167,7 @@ const ExportSchemaModal: React.FunctionComponent<{
         </Button>
         <Button
           onClick={() => {
-            /* TODO(COMPASS-8704) */
+            /* TODO(COMPASS-8704): download and track with trackSchemaExported */
           }}
           variant="primary"
         >
@@ -183,6 +187,7 @@ export default connect(
     exportedSchema: state.schemaExport.exportedSchema,
   }),
   {
+    onExportedSchemaCopied: trackSchemaExported,
     onCancelSchemaExport: cancelExportSchema,
     onChangeSchemaExportFormat: changeExportSchemaFormat,
     onClose: closeExportSchema,

packages/compass-schema/src/stores/schema-export-reducer.ts

@@ -195,6 +195,22 @@ const _trackSchemaExported = ({
   };
 };
 
+export const onSchemaExportCopied = (): SchemaThunkAction<void> => {
+  return (dispatch, getState) => {
+    const {
+      schemaAnalysis: { schema },
+      schemaExport: { exportFormat },
+    } = getState();
+    dispatch(
+      _trackSchemaExported({
+        schema,
+        format: exportFormat,
+        source: 'schema_tab',
+      })
+    );
+  };
+};
+
 export const changeExportSchemaFormat = (
   exportFormat: SchemaFormat
 ): SchemaThunkAction<
@@ -228,7 +244,7 @@ export const changeExportSchemaFormat = (
       }
     );
 
-    const { schemaAccessor, schema } = getState().schemaAnalysis;
+    const { schemaAccessor } = getState().schemaAnalysis;
 
     let exportedSchema: string;
     try {
@@ -265,14 +281,6 @@ export const changeExportSchemaFormat = (
       return;
     }
 
-    dispatch(
-      _trackSchemaExported({
-        schema,
-        format: exportFormat,
-        source: 'schema_tab',
-      })
-    );
-
     log.info(
       mongoLogId(1_001_000_344),
       'Schema export formatting complete',