Skip to content

Commit a0b79e0

Browse files
committed
WIP
1 parent 8f9d83f commit a0b79e0

File tree

1 file changed

+8
-11
lines changed

1 file changed

+8
-11
lines changed

packages/compass-collection/src/components/collection-header-actions/collection-header-actions.tsx

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -80,21 +80,18 @@ const CollectionHeaderActions: React.FunctionComponent<
8080
mockDataGeneratorAssignment?.assignment?.assignmentData?.variant ===
8181
ExperimentTestGroup.mockDataGeneratorVariant;
8282

83-
// Determine if we should show the Mock Data Generator button
8483
const shouldShowMockDataButton =
8584
isInMockDataTreatmentVariant &&
8685
atlasMetadata && // Only show in Atlas
87-
!isReadonly && // Don't show for readonly collections (views) // TODO is this redundant?
88-
!sourceName; // Don't show for views (sourceName indicates it's a view)
86+
!isReadonly && // Don't show for readonly collections (views)
87+
!sourceName; // sourceName indicates it's a view
8988

90-
// For now, we'll assume collection has data (no schema analysis dependency)
91-
// In the future, this will be replaced with actual schema analysis
92-
const hasCollectionData = true;
89+
const hasData = true; // TODO: CLOUDP-337090
9390

9491
// Determine if button should be enabled or disabled with tooltip
95-
const canGenerateMockData = hasCollectionData; // TODO: Redundant
96-
const disabledTooltipText = !hasCollectionData
97-
? 'Collection is empty'
92+
const isMockDataButtonEnabled = hasData; // TODO: CLOUDP-337090: also filter out overly nested collections
93+
const disabledTooltipText = !hasData
94+
? 'Please add data to your collection to generate similar mock documents'
9895
: undefined;
9996

10097
return (
@@ -166,13 +163,13 @@ const CollectionHeaderActions: React.FunctionComponent<
166163
)}
167164
{shouldShowMockDataButton && (
168165
<Tooltip
169-
enabled={!canGenerateMockData && !!disabledTooltipText}
166+
enabled={!isMockDataButtonEnabled && !!disabledTooltipText}
170167
trigger={
171168
<div>
172169
<Button
173170
data-testid="collection-header-generate-mock-data"
174171
size={ButtonSize.Small}
175-
disabled={!canGenerateMockData}
172+
disabled={!isMockDataButtonEnabled}
176173
onClick={onOpenMockDataModal}
177174
leftGlyph={<Icon glyph="Sparkle" />}
178175
>

0 commit comments

Comments
 (0)