Revert "Show some known OP replies in Following (
#5049)" (
#5301)
This reverts commit bff6aedecf007a4225bf30e1800933a9005cb0b2.
1 files changed, 6 insertions, 21 deletions
diff --git a/src/lib/api/feed-manip.ts b/src/lib/api/feed-manip.ts
index eaa760b4b..0eca8b165 100644
--- a/src/lib/api/feed-manip.ts
+++ b/src/lib/api/feed-manip.ts
@@ -379,7 +379,11 @@ export class FeedTuner {
): FeedViewPostsSlice[] => {
for (let i = 0; i < slices.length; i++) {
const slice = slices[i]
- if (slice.isReply && !shouldDisplayReplyInFollowing(slice, userDid)) {
+ if (
+ slice.isReply &&
+ !slice.isRepost &&
+ !shouldDisplayReplyInFollowing(slice.getAuthors(), userDid)
+ ) {
slices.splice(i, 1)
i--
}
@@ -443,13 +447,9 @@ function areSameAuthor(authors: AuthorContext): boolean {
}
function shouldDisplayReplyInFollowing(
- slice: FeedViewPostsSlice,
+ authors: AuthorContext,
userDid: string,
): boolean {
- if (slice.isRepost) {
- return true
- }
- const authors = slice.getAuthors()
const {author, parentAuthor, grandparentAuthor, rootAuthor} = authors
if (!isSelfOrFollowing(author, userDid)) {
// Only show replies from self or people you follow.
@@ -463,21 +463,6 @@ function shouldDisplayReplyInFollowing(
// Always show self-threads.
return true
}
- if (
- parentAuthor &&
- parentAuthor.did !== author.did &&
- rootAuthor &&
- rootAuthor.did === author.did &&
- slice.items.length > 2
- ) {
- // If you follow A, show A -> someone[>0 likes] -> A chains too.
- // This is different from cases below because you only know one person.
- const parentPost = slice.items[1].post
- const parentLikeCount = parentPost.likeCount ?? 0
- if (parentLikeCount > 0) {
- return true
- }
- }
// From this point on we need at least one more reason to show it.
if (
parentAuthor &&
|