about summary refs log tree commit diff
path: root/src/state/feed-feedback.tsx
Commit message (Collapse)AuthorAgeFilesLines
* Disambiguate feed component naming (#7040)dan2024-12-101-1/+1
| | | | | * Rename posts/Feed* -> posts/PostFeed* * Rename notifications/Feed* -> notifications/NotificationFeed*
* [Statsig] Remove client downsampling (#6153)Hailey2024-11-151-3/+3
|
* Lower feedfeedback seen threshold to 0.5s (#6127)dan2024-11-051-1/+1
|
* Remove .withProxy() calls (#4929)dan2024-08-131-31/+15
|
* Show replies in context of their threads (#4871)dan2024-08-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Don't reconstruct threads from separate posts * Remove post-level dedupe for now * Change repost dedupe condition to look just at length * Delete unused isThread * Delete another isThread field It is now meaningless because there's nothing special about author threads. * Narrow down slice item shape so it does not need reply * Consolidate slice validation criteria in one place * Show replies in context * Make fallback marker work * Remove misleading and now-unused property It was called rootUri but it was actually the leaf URI. Regardless, it's not used anymore. * Add by-thread dedupe to non-author feeds * Add post-level dedupe * Always count from the start This is easier to think about. * Only tuner state need to be untouched on dry run * Account for threads in reply filtering * Remove repost deduping This is already being taken care of by item-level deduping. It's also now wrong and removing too much (since it wasn't filtering for reposts directly). * Calculate rootUri correctly * Apply Following settings to all lists * Don't dedupe intentional reposts by thread * Show reply parent when ambiguous * Explicitly remove orphaned replies from following/lists * Fix thread dedupe to work across pages * Mark grandparent-blocked as orphaned * Guard tuner state change by dryRun * Remove dead code * Don't dedupe feedgen threads * Revert "Apply Following settings to all lists" This reverts commit aff86be6d37b60cc5d0ac38f22c31a4808342cf4. Let's not do this yet and have a bit more discussion. This is a chunky change already. * Reason belongs to a slice, not item * Logically feedContext belongs to the slice * Update comment to reflect latest behavior
* [Session] Experiment: Don't use withProxy (#4762)dan2024-07-101-13/+31
| | | | | * Reorder statements * Remove withProxy() usage behind a gate
* Fix feed feedback (#4730)dan2024-07-041-7/+5
|
* [Statsig] Send Discover aggregate interactions (#4599)dan2024-06-221-1/+106
|
* Replace getAgent() with reading agent (#4243)dan2024-05-281-3/+3
| | | | | * Replace getAgent() with agent * Replace {agent} with agent
* Implement FeedFeedback API (#3498)Paul Frazee2024-05-071-0/+151
* Implement onViewableItemsChanged on List.web.tsx * Introduce onItemSeen to List API * Add FeedFeedback tracker * Add clickthrough interaction tracking * Add engagement interaction tracking * Reduce duplicate sends, introduce a flushAndReset to be triggered on refreshes, and modify the api design a bit * Wire up SDK types and feedContext * Avoid needless function allocations * Fix schema usage * Add show more / show less buttons * Fix minor rendering issue on mobile menu * Wire up sendInteractions() * Fix logic error * Fix: it's item not uri * Update 'seen' to mean 3 seconds on-screen with some significant portion visible * Fix non-reactive debounce * Move methods out * Use a WeakSet for deduping * Reset timeout * 3 -> 2 seconds * Oopsie * Throttle instead * Fix divider * Remove explicit flush calls * Rm unused --------- Co-authored-by: dan <dan.abramov@gmail.com>