Skip to content

Commit

Permalink
fix(lld): define optOut as disabled share personalized recommendations
Browse files Browse the repository at this point in the history
  • Loading branch information
thesan committed Feb 13, 2025
1 parent 8e3ff77 commit e2228e8
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,8 @@ describe("LNSBannerCard", () => {
render(<LNSBannerCard location={location} fallback={<div data-testid="fallback" />} />, {
initialState: {
settings: {
shareAnalytics: isOptIn,
shareAnalytics: true,
sharePersonalizedRecommandations: isOptIn,
overriddenFeatureFlags: {
lldNanoSUpsellBanners: { enabled: ffEnabled, params: ffParams },
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,8 @@ describe("LNSNotificationBanner", () => {
render(<LNSNotificationBanner />, {
initialState: {
settings: {
shareAnalytics: isOptIn,
shareAnalytics: true,
sharePersonalizedRecommandations: isOptIn,
overriddenFeatureFlags: {
lldNanoSUpsellBanners: { enabled: ffEnabled, params: ffParams },
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ import { useSelector } from "react-redux";
import { useFeature } from "@ledgerhq/live-common/featureFlags/index";
import { track } from "~/renderer/analytics/segment";
import { openURL } from "~/renderer/linking";
import { trackingEnabledSelector } from "~/renderer/reducers/settings";
import { sharePersonalizedRecommendationsSelector } from "~/renderer/reducers/settings";
import { AnalyticsButton, AnalyticsPage, type LNSBannerLocation } from "../types";

export function useLNSBanner(location: LNSBannerLocation) {
const isOptIn = useSelector(trackingEnabledSelector);
const isOptIn = useSelector(sharePersonalizedRecommendationsSelector);
const ff = useFeature("lldNanoSUpsellBanners");
const params = ff?.params?.[isOptIn ? "opted_in" : "opted_out"];

Expand Down

0 comments on commit e2228e8

Please sign in to comment.