Skip to content

Commit 3b671a0

Browse files
ehoops-czclaude
andauthored
feat: remove BYOD feature flag, make AI Workspace callouts visible to all users (#7682)
Co-authored-by: Claude <[email protected]>
1 parent 5dd0db8 commit 3b671a0

File tree

5 files changed

+11
-34
lines changed

5 files changed

+11
-34
lines changed

frontend/src/common/featureFlags/features.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,4 @@ export enum FEATURES {
22
GENE_SETS = "gs",
33
CURATOR = "curator",
44
FILTER = "filter",
5-
BYOD = "byod",
65
}

frontend/src/components/BYODConnectors/BYODCallout/index.tsx

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,11 @@
11
import React from "react";
22
import { StyledButton } from "../style";
33
import { useBYODModal } from "src/contexts/BYODModalContext";
4-
import { useFeatureFlag } from "src/common/hooks/useFeatureFlag";
5-
import { FEATURES } from "src/common/featureFlags/features";
64
import SparkleIcon from "src/common/images/sparkle-icon.svg";
75
import { CalloutTextWrapper, CalloutTitle, StyledCallout } from "./style";
86

97
const BYODCallout: React.FC = () => {
108
const { openModal } = useBYODModal();
11-
const isBYODEnabled = useFeatureFlag(FEATURES.BYOD);
12-
13-
if (!isBYODEnabled) return null;
149

1510
return (
1611
<StyledCallout intent="info" icon={<SparkleIcon />}>

frontend/src/components/BYODConnectors/BYODToast/index.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@ import { StyledButton } from "../style";
55
import { TOAST_BUTTON_TEXT } from "../constants";
66
import { dismissToast, isToastDismissed } from "./utils";
77
import { useBYODModal } from "src/contexts/BYODModalContext";
8-
import { useFeatureFlag } from "src/common/hooks/useFeatureFlag";
9-
import { FEATURES } from "src/common/featureFlags/features";
108
import SparkleIcon from "src/common/images/sparkle-icon.svg";
119

1210
const BYOD_TOAST_ID = "byod-ai-workspace";
@@ -16,7 +14,6 @@ const TOAST_MESSAGE =
1614
export default function BYODToast(): JSX.Element | null {
1715
const [isVisible, setIsVisible] = useState(false);
1816
const { openModal } = useBYODModal();
19-
const isBYODEnabled = useFeatureFlag(FEATURES.BYOD);
2017

2118
useEffect(() => {
2219
setIsVisible(!isToastDismissed(BYOD_TOAST_ID));
@@ -31,7 +28,7 @@ export default function BYODToast(): JSX.Element | null {
3128
openModal();
3229
};
3330

34-
if (!isBYODEnabled || !isVisible) return null;
31+
if (!isVisible) return null;
3532

3633
return (
3734
<StyledToast>

frontend/src/components/Header/index.tsx

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ import { FC } from "react";
33
import { HomepageLink } from "../common/HomepageLink";
44
import AuthButtons from "./components/AuthButtons";
55
import Nav from "src/components/Header/components/Nav";
6-
import { useFeatureFlag } from "src/common/hooks/useFeatureFlag";
7-
import { FEATURES } from "src/common/featureFlags/features";
86
import { useBYODModal } from "src/contexts/BYODModalContext";
97
import { useConnect } from "./connect";
108
import {
@@ -29,7 +27,6 @@ const Header: FC = () => {
2927
} = useConnect();
3028

3129
const { openModal } = useBYODModal();
32-
const isBYODEnabled = useFeatureFlag(FEATURES.BYOD);
3330

3431
const handleBYODClick = () => {
3532
openModal();
@@ -43,15 +40,9 @@ const Header: FC = () => {
4340
<Nav pathname={pathname} />
4441
</Left>
4542
<Right>
46-
{isBYODEnabled && (
47-
<Button
48-
sdsType="primary"
49-
sdsStyle="square"
50-
onClick={handleBYODClick}
51-
>
52-
Explore Your Data
53-
</Button>
54-
)}
43+
<Button sdsType="primary" sdsStyle="square" onClick={handleBYODClick}>
44+
Explore Your Data
45+
</Button>
5546

5647
<StyledInputDropdown
5748
disabled={false}

frontend/src/components/MobileFriendlyHeader/index.tsx

Lines changed: 7 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,6 @@ import { track } from "src/common/analytics";
55
import { EVENTS } from "src/common/analytics/events";
66
import { ROUTES } from "src/common/constants/routes";
77
import AuthButtons from "src/components/Header/components/AuthButtons";
8-
import { useFeatureFlag } from "src/common/hooks/useFeatureFlag";
9-
import { FEATURES } from "src/common/featureFlags/features";
108
import { useBYODModal } from "src/contexts/BYODModalContext";
119
import { HomepageLink } from "../common/HomepageLink";
1210
import {
@@ -55,7 +53,6 @@ const LandingHeader: FC<Props> = ({
5553
const mobileNavTray = useRef<HTMLDivElement>(null);
5654
const [mobileMenuOpen, setMobileMenuOpen] = useState(false);
5755
const { openModal } = useBYODModal();
58-
const isBYODEnabled = useFeatureFlag(FEATURES.BYOD);
5956

6057
const handleBYODClick = () => {
6158
openModal();
@@ -156,15 +153,13 @@ const LandingHeader: FC<Props> = ({
156153
<Nav pathname={pathname} />
157154
</Left>
158155
<Right>
159-
{isBYODEnabled && (
160-
<Button
161-
sdsType="primary"
162-
sdsStyle="square"
163-
onClick={handleBYODClick}
164-
>
165-
Explore Your Data
166-
</Button>
167-
)}
156+
<Button
157+
sdsType="primary"
158+
sdsStyle="square"
159+
onClick={handleBYODClick}
160+
>
161+
Explore Your Data
162+
</Button>
168163

169164
<LinkWrapper isActive={isRouteActive(pathname, ROUTES.DOCS)}>
170165
<a

0 commit comments

Comments
 (0)