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

Implement separate mod activities for feature, lock post #2716

Merged
merged 3 commits into from
Feb 18, 2023

Conversation

Nutomic
Copy link
Member

@Nutomic Nutomic commented Feb 8, 2023

Also includes collection for featured posts. Later we also need to do the same for Comment.distinguished

Copy link
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Outside of the DB stuff it looks good.

I leave it up to you whether you want to do this, but it might be good to add federation tests, to make sure adding and removing mods, and locking and unlocking federated posts is working. (I think they already exist for the post locking, but maybe not for adding / removing mods).

crates/apub/src/http/community.rs Show resolved Hide resolved
crates/db_schema/src/impls/community.rs Show resolved Hide resolved
crates/db_schema/src/impls/community.rs Show resolved Hide resolved
crates/db_schema/src/impls/post.rs Show resolved Hide resolved
docker/dev/Dockerfile Outdated Show resolved Hide resolved
migrations/2023-02-07-030958_community-collections/up.sql Outdated Show resolved Hide resolved
@dessalines dessalines enabled auto-merge (squash) February 13, 2023 16:48
@dessalines
Copy link
Member

Failing clippy.

Also includes collection for featured posts. Later we also need
to do the same for Comment.distinguished
@dessalines dessalines merged commit 62663a9 into main Feb 18, 2023
Nutomic added a commit that referenced this pull request Feb 20, 2023
* Implement separate mod activities for feature, lock post

Also includes collection for featured posts. Later we also need
to do the same for Comment.distinguished

* some changes

---------

Co-authored-by: Dessalines <dessalines@users.noreply.github.com>
dessalines added a commit that referenced this pull request Jun 7, 2023
* Remove deprecated field `group.moderators`

* Drop column local_site.federation_debug

* Remove backwards compatibility code for feature/lock post

#2716

---------

Co-authored-by: Dessalines <dessalines@users.noreply.github.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.

None yet

2 participants