Add feature gate

This commit is contained in:
Eric Bailey
2025-01-17 16:48:40 -06:00
parent fa7f422aa2
commit b4bf7fd919
4 changed files with 48 additions and 28 deletions
+1
View File
@@ -7,3 +7,4 @@ export type Gate =
| 'test_gate_1'
| 'test_gate_2'
| 'trending_topics_beta'
| 'yolo'
@@ -1,9 +1,11 @@
import React from 'react'
import {msg, Trans} from '@lingui/macro'
import {useLingui} from '@lingui/react'
import {NativeStackScreenProps} from '@react-navigation/native-stack'
import {CommonNavigatorParams} from '#/lib/routes/types'
import {logEvent} from '#/lib/statsig/statsig'
import {useGate} from '#/lib/statsig/statsig'
import {isNative} from '#/platform/detection'
import {useAutoplayDisabled, useSetAutoplayDisabled} from '#/state/preferences'
import {
@@ -40,6 +42,10 @@ export function ContentAndMediaSettingsScreen({}: Props) {
const {trendingDisabled, trendingVideoDisabled} = useTrendingSettings()
const {setTrendingDisabled, setTrendingVideoDisabled} =
useTrendingSettingsApi()
const gate = useGate()
const areVideoFeedsEnabled = React.useMemo(() => {
return gate('yolo')
}, [gate])
return (
<Layout.Screen>
@@ -139,27 +145,29 @@ export function ContentAndMediaSettingsScreen({}: Props) {
<Toggle.Platform />
</SettingsList.Item>
</Toggle.Item>
<Toggle.Item
name="show_trending_videos"
label={_(msg`Enable trending videos`)}
value={!trendingVideoDisabled}
onChange={value => {
const hide = Boolean(!value)
if (hide) {
logEvent('trendingVideos:hide', {context: 'settings'})
} else {
logEvent('trendingVideos:show', {context: 'settings'})
}
setTrendingVideoDisabled(hide)
}}>
<SettingsList.Item>
<SettingsList.ItemIcon icon={Graph} />
<SettingsList.ItemText>
<Trans>Enable trending videos</Trans>
</SettingsList.ItemText>
<Toggle.Platform />
</SettingsList.Item>
</Toggle.Item>
{areVideoFeedsEnabled && (
<Toggle.Item
name="show_trending_videos"
label={_(msg`Enable trending videos`)}
value={!trendingVideoDisabled}
onChange={value => {
const hide = Boolean(!value)
if (hide) {
logEvent('trendingVideos:hide', {context: 'settings'})
} else {
logEvent('trendingVideos:show', {context: 'settings'})
}
setTrendingVideoDisabled(hide)
}}>
<SettingsList.Item>
<SettingsList.ItemIcon icon={Graph} />
<SettingsList.ItemText>
<Trans>Enable trending videos</Trans>
</SettingsList.ItemText>
<Toggle.Platform />
</SettingsList.Item>
</Toggle.Item>
)}
</>
)}
</SettingsList.Container>
+13 -5
View File
@@ -21,6 +21,7 @@ import {
} from '#/lib/constants'
import {useInitialNumToRender} from '#/lib/hooks/useInitialNumToRender'
import {useWebMediaQueries} from '#/lib/hooks/useWebMediaQueries'
import {useGate} from '#/lib/statsig/statsig'
import {logEvent} from '#/lib/statsig/statsig'
import {useTheme} from '#/lib/ThemeContext'
import {logger} from '#/logger'
@@ -188,7 +189,13 @@ let PostFeed = ({
const lastFetchRef = React.useRef<number>(Date.now())
const [feedType, feedUri, feedTab] = feed.split('|')
const {gtMobile, gtTablet} = useBreakpoints()
const isVideoFeed = feedUri === VIDEO_FEED_URI
const gate = useGate()
const areVideoFeedsEnabled = React.useMemo(() => {
return isNative && gate('yolo')
}, [gate])
const isVideoFeedAndIsEnabled = React.useMemo(() => {
return feedUri === VIDEO_FEED_URI && areVideoFeedsEnabled
}, [feedUri, areVideoFeedsEnabled])
const opts = React.useMemo(
() => ({enabled, ignoreFilterFor}),
@@ -329,7 +336,7 @@ let PostFeed = ({
} else if (data) {
let sliceIndex = -1
if (isVideoFeed && isNative) {
if (isVideoFeedAndIsEnabled) {
const rows: FeedPostSliceItem[][] = []
let slices: {slice: FeedPostSlice; index: number}[] = []
for (const page of data.pages) {
@@ -386,7 +393,7 @@ let PostFeed = ({
})
}
} else if (sliceIndex === 15) {
if (isNative && !trendingVideoDisabled) {
if (areVideoFeedsEnabled && !trendingVideoDisabled) {
arr.push({
type: 'interstitialTrendingVideos',
key: 'interstitial-' + sliceIndex + '-' + lastFetchedAt,
@@ -461,7 +468,7 @@ let PostFeed = ({
})
}
} else {
if (isVideoFeed && isNative) {
if (isVideoFeedAndIsEnabled) {
arr.push({
type: 'videoGridRowPlaceholder',
key: 'videoGridRowPlaceholder',
@@ -490,7 +497,8 @@ let PostFeed = ({
trendingVideoDisabled,
gtTablet,
gtMobile,
isVideoFeed,
isVideoFeedAndIsEnabled,
areVideoFeedsEnabled,
])
// events
+5 -2
View File
@@ -10,9 +10,10 @@ import {
import {msg, Trans} from '@lingui/macro'
import {useLingui} from '@lingui/react'
import {useGate} from '#/lib/statsig/statsig'
import {cleanError} from '#/lib/strings/errors'
import {logger} from '#/logger'
import {isNative,isWeb} from '#/platform/detection'
import {isNative, isWeb} from '#/platform/detection'
import {useModerationOpts} from '#/state/preferences/moderation-opts'
import {useGetPopularFeedsQuery} from '#/state/queries/feed'
import {usePreferencesQuery} from '#/state/queries/preferences'
@@ -308,6 +309,7 @@ export function Explore() {
error: feedsError,
fetchNextPage: fetchNextFeedsPage,
} = useGetPopularFeedsQuery({limit: 10})
const gate = useGate()
const isLoadingMoreProfiles = isFetchingNextProfilesPage && !isLoadingProfiles
const onLoadMoreProfiles = React.useCallback(async () => {
@@ -348,7 +350,7 @@ export function Explore() {
key: `trending-topics`,
})
if (isNative) {
if (isNative && gate('yolo')) {
i.push({
type: 'trendingVideos',
key: `trending-videos`,
@@ -508,6 +510,7 @@ export function Explore() {
preferencesError,
hasNextProfilesPage,
hasNextFeedsPage,
gate,
])
const renderItem = React.useCallback(