Skip to content

Commit f87ec6d

Browse files
committed
chore: Remove feature flag PrivacyProductsBundle
1 parent 31c2a4c commit f87ec6d

File tree

6 files changed

+11
-70
lines changed

6 files changed

+11
-70
lines changed

functional-tests/fixtures/baseTest.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ export const defaultLocalForcedFeatureFlags: FeatureFlagName[] = [
2222
"EditScanProfileDetails",
2323
"SubPlat3",
2424
"Announcements",
25-
"PrivacyProductsBundle",
2625
];
2726

2827
const test = baseTest.extend<{

src/app/(bundle)/bundle/onboarding/BundleOnboardingView.stories.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,5 @@ type Story = StoryObj<typeof BundleOnboardingView>;
1717
export const BundleOnboarding: Story = {
1818
args: {
1919
l10n: getL10n(),
20-
enabledFeatureFlags: ["PrivacyProductsBundle"],
2120
},
2221
};

src/app/(bundle)/bundle/onboarding/page.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import {
1111
} from "../../../functions/l10n/serverComponents";
1212
import { BundleOnboardingView } from "./BundleOnboardingView";
1313
import { getEnabledFeatureFlags } from "../../../../db/tables/featureFlags";
14-
import NotFound from "../../../not-found";
1514
import { getServerSession } from "../../../functions/server/getServerSession";
1615

1716
export default async function Page() {
@@ -24,9 +23,7 @@ export default async function Page() {
2423
}
2524
: { isSignedOut: true },
2625
);
27-
if (!enabledFeatureFlags.includes("PrivacyProductsBundle")) {
28-
return NotFound();
29-
}
26+
3027
const headersList = await headers();
3128
const countryCode = getCountryCode(headersList);
3229
const l10n = getL10n(await getAcceptLangHeaderInServerComponents());

src/app/(proper_react)/(redesign)/(public)/LandingViewRedesign/LandingViewRedesign.stories.tsx

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -92,16 +92,6 @@ export const LandingRedesignUsScanLimit: Story = {
9292
},
9393
};
9494

95-
export const LandingRedesignUsWithPrivacyProductBundle: Story = {
96-
name: "US visitors - With Privacy Product Bundle",
97-
args: {
98-
eligibleForPremium: true,
99-
countryCode: "us",
100-
scanLimitReached: false,
101-
enabledFeatureFlags: ["LandingPageRedesign", "PrivacyProductsBundle"],
102-
},
103-
};
104-
10595
export const LandingRedesignUsDisableOneRepScans: Story = {
10696
name: "US visitors - Disable OneRep Scans",
10797
args: {
@@ -111,13 +101,3 @@ export const LandingRedesignUsDisableOneRepScans: Story = {
111101
enabledFeatureFlags: ["DisableOneRepScans"],
112102
},
113103
};
114-
115-
export const LandingRedesignUsScanLimitWithPrivacyProductBundle: Story = {
116-
name: "US visitors - Scan limit reached and with Privacy Product Bundle",
117-
args: {
118-
eligibleForPremium: true,
119-
countryCode: "us",
120-
scanLimitReached: true,
121-
enabledFeatureFlags: ["LandingPageRedesign", "PrivacyProductsBundle"],
122-
},
123-
};

src/app/(proper_react)/(redesign)/(public)/LandingViewRedesign/LandingViewRedesign.test.tsx

Lines changed: 10 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@ import Meta, {
1818
LandingRedesignUs,
1919
LandingRedesignUsDisableOneRepScans,
2020
LandingRedesignUsScanLimit,
21-
LandingRedesignUsScanLimitWithPrivacyProductBundle,
22-
LandingRedesignUsWithPrivacyProductBundle,
2321
} from "./LandingViewRedesign.stories";
2422
import { useTelemetry as useTelemetryImported } from "../../../../hooks/useTelemetry";
2523
import { deleteAllCookies } from "../../../../functions/client/deleteAllCookies";
@@ -260,21 +258,15 @@ describe("FAQ", () => {
260258

261259
describe("Pricing plan", () => {
262260
it("passes the axe accessibility test suite", async () => {
263-
const ComposedLanding = composeStory(
264-
LandingRedesignUsWithPrivacyProductBundle,
265-
Meta,
266-
);
261+
const ComposedLanding = composeStory(LandingRedesignUs, Meta);
267262
const { container } = render(<ComposedLanding />);
268263
expect(await axe(container)).toHaveNoViolations();
269264
}, 10_000);
270265

271266
it("can initiate sign in from the Monitor (free) pricing card", async () => {
272267
const user = userEvent.setup();
273268

274-
const ComposedDashboard = composeStory(
275-
LandingRedesignUsWithPrivacyProductBundle,
276-
Meta,
277-
);
269+
const ComposedDashboard = composeStory(LandingRedesignUs, Meta);
278270
render(<ComposedDashboard />);
279271
const freeCard = screen.getByLabelText("Monitor");
280272

@@ -290,10 +282,7 @@ describe("Pricing plan", () => {
290282

291283
it("counts the number of clicks on the pricing plan free tier button", async () => {
292284
const mockedRecord = useTelemetry();
293-
const ComposedDashboard = composeStory(
294-
LandingRedesignUsWithPrivacyProductBundle,
295-
Meta,
296-
);
285+
const ComposedDashboard = composeStory(LandingRedesignUs, Meta);
297286
render(<ComposedDashboard />);
298287

299288
const user = userEvent.setup();
@@ -314,10 +303,7 @@ describe("Pricing plan", () => {
314303

315304
it("counts the number of clicks on the pricing plan upsell button", async () => {
316305
const mockedRecord = useTelemetry();
317-
const ComposedDashboard = composeStory(
318-
LandingRedesignUsWithPrivacyProductBundle,
319-
Meta,
320-
);
306+
const ComposedDashboard = composeStory(LandingRedesignUs, Meta);
321307
render(<ComposedDashboard />);
322308

323309
const user = userEvent.setup();
@@ -342,10 +328,7 @@ describe("Pricing plan", () => {
342328

343329
it("counts the number of clicks on the pricing card Plus upsell button", async () => {
344330
const mockedRecord = useTelemetry();
345-
const ComposedDashboard = composeStory(
346-
LandingRedesignUsWithPrivacyProductBundle,
347-
Meta,
348-
);
331+
const ComposedDashboard = composeStory(LandingRedesignUs, Meta);
349332
render(<ComposedDashboard />);
350333

351334
const user = userEvent.setup();
@@ -370,10 +353,7 @@ describe("Pricing plan", () => {
370353

371354
it("counts the number of clicks on the pricing card free button", async () => {
372355
const mockedRecord = useTelemetry();
373-
const ComposedDashboard = composeStory(
374-
LandingRedesignUsWithPrivacyProductBundle,
375-
Meta,
376-
);
356+
const ComposedDashboard = composeStory(LandingRedesignUs, Meta);
377357
render(<ComposedDashboard />);
378358

379359
const user = userEvent.setup();
@@ -393,10 +373,7 @@ describe("Pricing plan", () => {
393373
});
394374

395375
it("confirms that the pricing card monthly upsell has the correct link for SubPlat2", async () => {
396-
const ComposedStory = composeStory(
397-
LandingRedesignUsWithPrivacyProductBundle,
398-
Meta,
399-
);
376+
const ComposedStory = composeStory(LandingRedesignUs, Meta);
400377
render(<ComposedStory />);
401378

402379
const plusCard = screen.getByLabelText("Monitor Plus");
@@ -415,17 +392,10 @@ describe("Pricing plan", () => {
415392
});
416393

417394
it("confirms that the pricing card monthly upsell has the correct link for SubPlat3", async () => {
418-
const ComposedStory = composeStory(
419-
LandingRedesignUsWithPrivacyProductBundle,
420-
Meta,
421-
);
395+
const ComposedStory = composeStory(LandingRedesignUs, Meta);
422396
render(
423397
<ComposedStory
424-
enabledFeatureFlags={[
425-
"LandingPageRedesign",
426-
"PrivacyProductsBundle",
427-
"SubPlat3",
428-
]}
398+
enabledFeatureFlags={["LandingPageRedesign", "SubPlat3"]}
429399
/>,
430400
);
431401

@@ -492,10 +462,7 @@ describe("Scan limit reached", () => {
492462
})),
493463
}),
494464
);
495-
const ComposedDashboard = composeStory(
496-
LandingRedesignUsScanLimitWithPrivacyProductBundle,
497-
Meta,
498-
);
465+
const ComposedDashboard = composeStory(LandingRedesignUsScanLimit, Meta);
499466
render(<ComposedDashboard />);
500467

501468
await waitFor(() => {

src/db/tables/featureFlags.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ export const featureFlagNames = [
7373
"EditScanProfileDetails",
7474
"SubPlat3",
7575
"Announcements",
76-
"PrivacyProductsBundle",
7776
"DisableOneRepScans",
7877
"DisableLandingToDashboardRedirect",
7978
"Moscary",

0 commit comments

Comments
 (0)