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

Fix handling of follows addressed to single value #2920

Merged
merged 4 commits into from Jun 6, 2023

Conversation

vpzomtrrfrt
Copy link
Contributor

Should fix follows from lotide, which seem to have been broken by #2829

Includes a deserialize_opt_one function in lemmy_apub, though maybe you want to move that to activitypub-federation with the others?

@Nutomic
Copy link
Member

Nutomic commented Jun 6, 2023

I would prefer to use deserialize_skip_error here as the value isnt necessary anyway. Also make the same change for Accept and UndoFollow.

Btw you need to run ./scripts/fix-clippy.sh

@Nutomic Nutomic enabled auto-merge (squash) June 6, 2023 21:52
@Nutomic Nutomic merged commit 09099e7 into LemmyNet:main Jun 6, 2023
1 check passed
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