Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rss: filter out replies server-side (tiny PR) #2518

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Conversation

bnewbold
Copy link
Contributor

Small efficiency enhancement, to filter out replies server-side. The main behavior change will be that RSS feeds have more actual posts in them (longer set), as we don't currently paginated until we have a full set, just filter out replies and reposts.

We'll also want to filter out reposts server-side, but that will require bluesky-social/atproto#2048

@bnewbold bnewbold requested a review from pfrazee January 12, 2024 22:55
@pfrazee pfrazee merged commit 1d3e202 into main Jan 15, 2024
4 checks passed
@pfrazee pfrazee deleted the bnewbold/rss-filter branch January 15, 2024 21:13
estrattonbailey added a commit that referenced this pull request Jan 18, 2024
* origin/main: (44 commits)
  Patch `@lingui/core` to fix `unraw` import resolution error (#2548)
  1.65
  Fix the fallback to discover behavior on the home feed (#2546)
  Bump android version code
  Bump ios build number
  Add a new home feed-api wrapper and give a header indicating the fallback behavior (#2534)
  Add accept-language header (#2457)
  rss: filter out replies server-side (#2518)
  feat: show muted/blocked status on list card (#2523)
  1.64 (#2521)
  Bump [email protected] (#2519)
  Create a profile record on new user (#2520)
  fix: truncate long email address (#2493)
  fix: set html lang according to app language (#2496)
  Ensure sessions and tokens are synced between tabs (#2498)
  package.json: cp --verbose doesn't exist on macos (#2501)
  Suggest post language correction (#2486)
  (optional) In app browser  (#2490)
  Toggle minimal shell on any scroll for web (#2499)
  ✨ New report type, appeal (#2455)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants