Skip to content

Commit 030bba7

Browse files
fix: make schemaComposition feature flag responsive (#5613)
* chore: remove useMemo for SchemaBrowserHeading * chore: move feature flag `schemaComposition` into SchemaBrowserHeading component
1 parent 185c027 commit 030bba7

File tree

2 files changed

+37
-36
lines changed

2 files changed

+37
-36
lines changed

src/dataExplorer/components/Schema.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ import {QueryScope} from 'src/shared/contexts/query'
2525

2626
// Utils
2727
import {getOrg} from 'src/organizations/selectors'
28-
import {isFlagEnabled} from 'src/shared/utils/featureFlag'
2928

3029
// Style
3130
import './Schema.scss'
@@ -82,9 +81,7 @@ const Schema: FC = () => {
8281
<div className="scroll--container">
8382
<DapperScrollbars>
8483
<div className="schema-browser" data-testid="schema-browser">
85-
{isFlagEnabled('schemaComposition') && (
86-
<SchemaBrowserHeading />
87-
)}
84+
<SchemaBrowserHeading />
8885
<BucketSelector />
8986
<div className="container-side-bar">
9087
<MeasurementSelector />

src/dataExplorer/components/SchemaBrowserHeading.tsx

Lines changed: 36 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, {FC, useContext, useMemo} from 'react'
1+
import React, {FC, useContext} from 'react'
22

33
// Components
44
import {
@@ -14,6 +14,9 @@ import SelectorTitle from 'src/dataExplorer/components/SelectorTitle'
1414
import {FluxQueryBuilderContext} from 'src/dataExplorer/context/fluxQueryBuilder'
1515
import {PersistanceContext} from 'src/dataExplorer/context/persistance'
1616

17+
// Utils
18+
import {isFlagEnabled} from 'src/shared/utils/featureFlag'
19+
1720
const FLUX_SYNC_DISABLE_TEXT = `Schema Sync is no longer available because the \
1821
code block has been edited.`
1922

@@ -43,38 +46,39 @@ const SchemaBrowserHeading: FC = () => {
4346
</div>
4447
)
4548

46-
return useMemo(
47-
() => (
48-
<FlexBox
49-
className="schema-browser-heading"
50-
justifyContent={JustifyContent.SpaceBetween}
51-
>
52-
<div className="schema-browser-heading--text">Schema Browser</div>
53-
<FlexBox className="flux-sync">
54-
<SlideToggle
55-
className="flux-sync--toggle"
56-
active={fluxSync}
57-
onChange={handleFluxSyncToggle}
58-
testID="flux-sync--toggle"
59-
disabled={disableToggle}
60-
tooltipText={disableTooltipText}
61-
/>
62-
<InputLabel className="flux-sync--label">
63-
<div
64-
className={`${disableToggle ? 'disabled' : ''}`}
65-
title={disableTooltipText}
66-
>
67-
<SelectorTitle
68-
label="Flux Sync"
69-
tooltipContents={tooltipContents}
70-
icon={IconFont.Sync}
71-
/>
72-
</div>
73-
</InputLabel>
74-
</FlexBox>
49+
if (!isFlagEnabled('schemaComposition')) {
50+
return null
51+
}
52+
53+
return (
54+
<FlexBox
55+
className="schema-browser-heading"
56+
justifyContent={JustifyContent.SpaceBetween}
57+
>
58+
<div className="schema-browser-heading--text">Schema Browser</div>
59+
<FlexBox className="flux-sync">
60+
<SlideToggle
61+
className="flux-sync--toggle"
62+
active={fluxSync}
63+
onChange={handleFluxSyncToggle}
64+
testID="flux-sync--toggle"
65+
disabled={disableToggle}
66+
tooltipText={disableTooltipText}
67+
/>
68+
<InputLabel className="flux-sync--label">
69+
<div
70+
className={`${disableToggle ? 'disabled' : ''}`}
71+
title={disableTooltipText}
72+
>
73+
<SelectorTitle
74+
label="Flux Sync"
75+
tooltipContents={tooltipContents}
76+
icon={IconFont.Sync}
77+
/>
78+
</div>
79+
</InputLabel>
7580
</FlexBox>
76-
),
77-
[fluxSync, toggleFluxSync]
81+
</FlexBox>
7882
)
7983
}
8084

0 commit comments

Comments
 (0)