Make apub extension fields optional (ref #1220) #163

Merged
dessalines merged 1 commits from optional-apub-extensions into main 12 months ago
Owner

I dont think these fields are really necessary for Lemmy to work, so we shouldnt expect other projects to implement them. With category its a bit tricky to set a default, but I think this still makes sense, because we are thinking of removing the categories.

I tested this by setting the optional fields as None in the sending code, and all the tests still passed fine (except sticky/lock post).

I dont think these fields are really necessary for Lemmy to work, so we shouldnt expect other projects to implement them. With category its a bit tricky to set a default, but I think this still makes sense, because we are thinking of removing the categories. I tested this by setting the optional fields as None in the sending code, and all the tests still passed fine (except sticky/lock post).
nutomic added 2 commits 12 months ago
dessalines merged commit def8af7d8a into main 12 months ago
The pull request has been merged as def8af7d8a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.