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

Federate pinned entries #831

Merged
merged 21 commits into from
Jul 4, 2024
Merged

Federate pinned entries #831

merged 21 commits into from
Jul 4, 2024

Conversation

BentiGorlich
Copy link
Member

@BentiGorlich BentiGorlich commented Jun 16, 2024

This PR implements the federation of pinning and unpinning entries in magazines:

  • Lemmy compatibility
  • Mbin <-> Mbin compatibility
  • When a remote moderator on a local magazine pins/unpins an entry -> announce it to the subscribers
  • Modlog entry for when a post is pinned

- add endpoint to get the featured collection
- add outgoing messages to send collection updates to subscribed instances
- handle the featured collection in the `ActivityPubManager.updateMagazine` method
- implement incoming add/remove activities for the featured collection
@BentiGorlich BentiGorlich added enhancement New feature or request activitypub ActivityPub related issues backend Backend related issues and pull requests labels Jun 16, 2024
@BentiGorlich BentiGorlich self-assigned this Jun 16, 2024
- it got broken when expanding the `AddRemoveFactory`
- When a remote moderator pins or unpins an entry this will now be announced to all subscribers of the magazine
@BentiGorlich BentiGorlich marked this pull request as ready for review June 26, 2024 09:51
@ghost
Copy link

ghost commented Jun 29, 2024

testing this one next :-)

@BentiGorlich
Copy link
Member Author

Same :)

melroy89 and others added 3 commits June 29, 2024 17:54
ghost
ghost previously approved these changes Jun 30, 2024
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

i can't find any issues... APPROVED!

@BentiGorlich BentiGorlich merged commit 6297c2b into main Jul 4, 2024
7 checks passed
@BentiGorlich BentiGorlich deleted the new/federate-pinned-entries branch July 4, 2024 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
activitypub ActivityPub related issues backend Backend related issues and pull requests enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants