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: Tolerate missing status fields on veganism.social #911

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

nikclayton
Copy link
Contributor

veganism.social is doing something weird that injects statuses into the user's timeline that are missing the fields that are normally present on authenticated statuses.

So far I've noticed:

  • reblogged
  • favourited
  • bookmarked

on the top level Status are missing, and if the status contains a poll the voted field is also missing.

Cover up this breakage by setting defaults for these fields.

veganism.social is doing something weird that injects statuses into the
user's timeline that are missing the fields that are normally present
on authenticated statuses.

So far I've noticed:

- `reblogged`
- `favourited`
- `bookmarked`

on the top level `Status` are missing, and if the status contains a poll
the `voted` field is also missing.

Cover up this breakage by setting defaults for these fields.
@nikclayton nikclayton merged commit 344ba62 into main Aug 29, 2024
23 checks passed
@nikclayton nikclayton deleted the fix-veganism-social branch August 29, 2024 09:43
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.

1 participant