Skip to content

FEATURE: Add ActivityPub topic map, topic actions, post actions and topic info, and update ActivityPub post info #169

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

Merged
merged 7 commits into from
Feb 14, 2025

Conversation

angusmcleod
Copy link
Contributor

@pmusaraj Here's a new PR, with your fixes plus some additional checks, tests and perf improvements in d7db00a

pmusaraj and others added 3 commits February 13, 2025 13:40
…ons and topic info, and update ActivityPub post info. (discourse#161)" (discourse#167)

This reverts commit 05aa143.
…ibutes

Without these checks, non-AP topics can fail spectacularly. This is
still a work-in-progress, ideally we'd have better safeguards for this
in core.
@pmusaraj
Copy link
Contributor

Thanks @angusmcleod, appreciate it!

@angusmcleod
Copy link
Contributor Author

@pmusaraj Note that I moved the activity_pub_enabled topic setting into the topic extension: https://github.com/discourse/discourse-activity-pub/pull/169/files#diff-64ab3445e4463aa01b344f3521034cd345534bac676c434b465cdd9c5ea40e1eR13

@pmusaraj
Copy link
Contributor

Yeah, I made a mistake committing that, fixing it. Thanks!

@pmusaraj pmusaraj merged commit cd160b8 into discourse:main Feb 14, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants