diff options
author | Hailey <me@haileyok.com> | 2024-11-15 11:40:43 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-11-15 11:40:43 -0800 |
commit | 3bd14371330abf902690dff9b3f0f2036ef472e5 (patch) | |
tree | 641df6c63773a7784d435add9f7ee80fd9bc97c9 /src/view | |
parent | 400c4322833d9cd4a6fb1e322c9ba844d85eff10 (diff) | |
download | voidsky-3bd14371330abf902690dff9b3f0f2036ef472e5.tar.zst |
[Statsig] Remove client downsampling (#6153)
Diffstat (limited to 'src/view')
-rw-r--r-- | src/view/com/feeds/FeedPage.tsx | 4 | ||||
-rw-r--r-- | src/view/com/pager/Pager.tsx | 6 | ||||
-rw-r--r-- | src/view/com/pager/Pager.web.tsx | 9 | ||||
-rw-r--r-- | src/view/com/posts/Feed.tsx | 4 | ||||
-rw-r--r-- | src/view/screens/Home.tsx | 9 |
5 files changed, 13 insertions, 19 deletions
diff --git a/src/view/com/feeds/FeedPage.tsx b/src/view/com/feeds/FeedPage.tsx index d61a81498..1028d7e64 100644 --- a/src/view/com/feeds/FeedPage.tsx +++ b/src/view/com/feeds/FeedPage.tsx @@ -74,7 +74,7 @@ export function FeedPage({ scrollToTop() truncateAndInvalidate(queryClient, FEED_RQKEY(feed)) setHasNew(false) - logEvent('feed:refresh:sampled', { + logEvent('feed:refresh', { feedType: feed.split('|')[0], feedUrl: feed, reason: 'soft-reset', @@ -98,7 +98,7 @@ export function FeedPage({ scrollToTop() truncateAndInvalidate(queryClient, FEED_RQKEY(feed)) setHasNew(false) - logEvent('feed:refresh:sampled', { + logEvent('feed:refresh', { feedType: feed.split('|')[0], feedUrl: feed, reason: 'load-latest', diff --git a/src/view/com/pager/Pager.tsx b/src/view/com/pager/Pager.tsx index 4d5da960c..aca3245a7 100644 --- a/src/view/com/pager/Pager.tsx +++ b/src/view/com/pager/Pager.tsx @@ -15,7 +15,7 @@ const AnimatedPagerView = Animated.createAnimatedComponent(PagerView) export interface PagerRef { setPage: ( index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], + reason: LogEvents['home:feedDisplayed']['reason'], ) => void } @@ -32,7 +32,7 @@ interface Props { onPageSelected?: (index: number) => void onPageSelecting?: ( index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], + reason: LogEvents['home:feedDisplayed']['reason'], ) => void onPageScrollStateChanged?: ( scrollState: 'idle' | 'dragging' | 'settling', @@ -61,7 +61,7 @@ export const Pager = forwardRef<PagerRef, React.PropsWithChildren<Props>>( React.useImperativeHandle(ref, () => ({ setPage: ( index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], + reason: LogEvents['home:feedDisplayed']['reason'], ) => { pagerView.current?.setPage(index) onPageSelecting?.(index, reason) diff --git a/src/view/com/pager/Pager.web.tsx b/src/view/com/pager/Pager.web.tsx index 2cce727c0..e6909fe10 100644 --- a/src/view/com/pager/Pager.web.tsx +++ b/src/view/com/pager/Pager.web.tsx @@ -18,7 +18,7 @@ interface Props { onPageSelected?: (index: number) => void onPageSelecting?: ( index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], + reason: LogEvents['home:feedDisplayed']['reason'], ) => void } export const Pager = React.forwardRef(function PagerImpl( @@ -38,17 +38,14 @@ export const Pager = React.forwardRef(function PagerImpl( React.useImperativeHandle(ref, () => ({ setPage: ( index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], + reason: LogEvents['home:feedDisplayed']['reason'], ) => { onTabBarSelect(index, reason) }, })) const onTabBarSelect = React.useCallback( - ( - index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], - ) => { + (index: number, reason: LogEvents['home:feedDisplayed']['reason']) => { const scrollY = window.scrollY // We want to determine if the tabbar is already "sticking" at the top (in which // case we should preserve and restore scroll), or if it is somewhere below in the diff --git a/src/view/com/posts/Feed.tsx b/src/view/com/posts/Feed.tsx index 905c1e0e0..d6cf6dac5 100644 --- a/src/view/com/posts/Feed.tsx +++ b/src/view/com/posts/Feed.tsx @@ -403,7 +403,7 @@ let Feed = ({ // = const onRefresh = React.useCallback(async () => { - logEvent('feed:refresh:sampled', { + logEvent('feed:refresh', { feedType: feedType, feedUrl: feed, reason: 'pull-to-refresh', @@ -421,7 +421,7 @@ let Feed = ({ const onEndReached = React.useCallback(async () => { if (isFetching || !hasNextPage || isError) return - logEvent('feed:endReached:sampled', { + logEvent('feed:endReached', { feedType: feedType, feedUrl: feed, itemCount: feedItems.length, diff --git a/src/view/screens/Home.tsx b/src/view/screens/Home.tsx index 237449383..cadfb4890 100644 --- a/src/view/screens/Home.tsx +++ b/src/view/screens/Home.tsx @@ -141,7 +141,7 @@ function HomeScreenReady({ useFocusEffect( useNonReactiveCallback(() => { if (selectedFeed) { - logEvent('home:feedDisplayed:sampled', { + logEvent('home:feedDisplayed', { index: selectedIndex, feedType: selectedFeed.split('|')[0], feedUrl: selectedFeed, @@ -163,12 +163,9 @@ function HomeScreenReady({ ) const onPageSelecting = React.useCallback( - ( - index: number, - reason: LogEvents['home:feedDisplayed:sampled']['reason'], - ) => { + (index: number, reason: LogEvents['home:feedDisplayed']['reason']) => { const feed = allFeeds[index] - logEvent('home:feedDisplayed:sampled', { + logEvent('home:feedDisplayed', { index, feedType: feed.split('|')[0], feedUrl: feed, |