-
-
Notifications
You must be signed in to change notification settings - Fork 866
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
Handle federated reports from Mastodon, Kbin #4323
Conversation
794c935
to
5f58690
Compare
/// Report reason as sent by Lemmy | ||
pub(crate) summary: Option<String>, | ||
/// Report reason as sent by Mastodon | ||
pub(crate) content: Option<String>, | ||
#[serde(rename = "type")] | ||
pub(crate) kind: FlagType, | ||
pub(crate) id: Url, | ||
pub(crate) audience: Option<ObjectId<ApubCommunity>>, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main changes here. Mastodon uses a different field for the reason, and the object is an array with user id and one or more post/comment ids, of which Lemmy can only handle one.
7aa0750
to
4301480
Compare
064e238
to
116f53f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once you get it passing and conflicts resolved, feel free to merge.
f36bb77
to
aff7a09
Compare
No description provided.