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

Blocks #922

Merged
merged 32 commits into from
Apr 29, 2023
Merged

Blocks #922

merged 32 commits into from
Apr 29, 2023

Conversation

dholms
Copy link
Collaborator

@dholms dholms commented Apr 28, 2023

Implements blocks as a new record type - symmetric to follows

Blocks are enforced on the server for most routes:

  • stripped out of feeds, search & aggregate views
  • annotated on getProfile in viewer state
  • throws when trying to getAuthorFeed

@dholms dholms marked this pull request as ready for review April 29, 2023 00:35
@dholms dholms merged commit db1042c into main Apr 29, 2023
@dholms dholms deleted the blocks branch April 29, 2023 00:36
sugyan added a commit to sugyan/atrium that referenced this pull request Apr 29, 2023
mloar pushed a commit to mloar/atproto that referenced this pull request Nov 15, 2023
* block lexicons

* table schema

* block processing

* Actor-block migration

* Add blocked post lexicon to getPostThread

* getBlocks routes

* Add missing lexicons to bsky

* Add block state to actor views

* throw on blocked accounts in getAuthorFeed

* blcok state in profile views

* Update getPostThread to support block views, update existing test

* strip out blocks out oof timeline & popular

* more routes

* Test blocked posts in threads

* Support blocks on record embeds

* notifs

* Update pds test snapshots w/ block state

* fix search

* Update pds actor search snapshots w/ blocks

* test

* tests

* update lex

* fix up from returning blocking as record

* ci for branch

* fix build issue in bsky

* more blocks tests

* fix actor search test

* hook getBlocks up to router

* bugfix & actually test lol

---------

Co-authored-by: Devin Ivy <devinivy@gmail.com>
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.

3 participants